Replace http with server #12

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@bclinkinbeard
Collaborator

Enables both require('atomify').server and require('atomify/server'). In either case, it just points to the same file the CLI uses, and allows config via the serve property of the object passed in.

The inconsistency between serve and server kinda bugs me. I think maybe we should rename the CLI flag and the config property to be server. What do you think?

@Techwraith
Collaborator

I agree with the rename atomify server sounds like a good enough command.

@bclinkinbeard
Collaborator

Cool, will do that and finish read me updates tomorrow. 1.0 is right around the corner!

On Thu, Feb 27, 2014 at 12:12 AM, Daniel Erickson
notifications@github.com wrote:

I agree with the rename atomify server sounds like a good enough command.

Reply to this email directly or view it on GitHub:
#12 (comment)

@Techwraith
Collaborator

<3

@bclinkinbeard
Collaborator

I manually merged this earlier. Published as 0.13.0

@bclinkinbeard bclinkinbeard deleted the server branch Feb 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment