Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

Assignment 1 #28

Closed
wants to merge 8 commits into from
Closed

Assignment 1 #28

wants to merge 8 commits into from

Conversation

rutaihwa
Copy link
Contributor

Basic server

@rutaihwa
Copy link
Contributor Author

Basic server, not very fancy. #1

method: 'GET',
path: '/version',
config:{

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove the empty lines here you are defining object.
And add spacing before function arguments and between : {

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @idanwe I am getting the hang of the style guide.

@hueniverse hueniverse added this to the 0.0.1 milestone Mar 16, 2015

// Routes
server.route({

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for empty line here.

@hueniverse hueniverse closed this Mar 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants