-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed the endpoint message to use app.name to match the endpoint route #76
Changes from 3 commits
9b9dc24
7e922d1
da18bcb
8bc43a8
289501a
5732256
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -128,7 +128,7 @@ var appServer = function(config) { | |
postRequest: self.config.postRequest | ||
}); | ||
|
||
var endpoint = path.posix.join(normalizedRoot, pkg.name); | ||
var endpoint = path.posix.join(normalizedRoot, app.name); | ||
self.log(" loaded app [" + pkg.name + "] at endpoint: " + endpoint); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you feel motivated, you could write a test for this using There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So I added the test, with a new entry in the "invalid_examples" folder, and added the assertion in the existing invalid examples test. I wasn't sure where it fit the best, so I'm happy to restructure if you think there's a better spot for it. |
||
}); | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be so kind to make this look similar to other lines? Everything is on the same line, there's a
-
, a period at the end, etc.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the formatting and added the PR link.