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

Assignment4 #131

Closed
wants to merge 4 commits into from
Closed

Assignment4 #131

wants to merge 4 commits into from

Conversation

rutaihwa
Copy link
Contributor

@rutaihwa rutaihwa commented May 4, 2015

Basic authentication #118 , assignment 4

internals.response = function (username) {

return '<h2>Welcome ' + username + '</h2>';

Copy link
Contributor

Choose a reason for hiding this comment

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

no need for newline here

@AdriVanHoudt
Copy link
Contributor

Linting results:
    lib/private.js:
        Line 42: space-before-function-paren - Missing space before function parentheses.
    test/private.js:
        Line 72: space-before-function-paren - Missing space before function parentheses.

@@ -19,6 +19,7 @@
"homepage": "https://github.com/hueniverse/hueniversity",
"dependencies": {
"hapi": "8.x.x",
"hapi-auth-basic": "^2.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

try using the 2.x.x notation for module versions

@hueniverse hueniverse modified the milestone: 0.0.4 May 12, 2015
@hueniverse hueniverse closed this May 12, 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