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

Create basic server #25

Closed
wants to merge 1 commit into from
Closed

Create basic server #25

wants to merge 1 commit into from

Conversation

ivanph
Copy link

@ivanph ivanph commented Mar 16, 2015

  • Adds basic server

Entry for assignment #1

@hueniverse hueniverse modified the milestone: 0.0.1 Mar 16, 2015
@@ -0,0 +1,3 @@
var Package = require('../package.json');

module.exports.number = Package.version;

Choose a reason for hiding this comment

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

This extra file doesn't add any value since all it does is load a file and exposes part of it. Better of to just require the package.json file directly. Also, version isn't a number :-)

@ivanph ivanph force-pushed the master branch 2 times, most recently from bc9a976 to a7b97cd Compare March 16, 2015 08:34
- Adds basic server

- Adds /version endpoint
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.

2 participants