Skip to content
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

Use js 'standard' formatter on pre-commit #172

Merged
merged 2 commits into from
Sep 21, 2015
Merged

Use js 'standard' formatter on pre-commit #172

merged 2 commits into from
Sep 21, 2015

Conversation

fhemberger
Copy link
Contributor

Closes #157 and #159.

@@ -8,7 +8,8 @@
"serve": "node build.js serve",
"lint": "eslint scripts build.js",
Copy link
Member

Choose a reason for hiding this comment

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

Remove the old lint script and explicit eslint dep below?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aye!

@phillipj
Copy link
Member

LGTM

@phillipj
Copy link
Member

.eslintrc could be deleted aswell

NOTICE: Don't reformat scripts in ./static (may be already minified)
@fhemberger
Copy link
Contributor Author

Removed .eslintrc.

`https://github.com/joyent/node/blob/${version}/ChangeLog`;
};
return semver.gte(version, '1.0.0')
? `https://github.com/nodejs/io.js/blob/${version}/CHANGELOG.md`
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be https://github.com/nodejs/node/blob/${version}/CHANGELOG.md?

@fhemberger
Copy link
Contributor Author

@nodejs/website Should we merge this?

@mikeal
Copy link
Contributor

mikeal commented Sep 21, 2015

+1

fhemberger added a commit that referenced this pull request Sep 21, 2015
Use js 'standard' formatter on pre-commit
@fhemberger fhemberger merged commit 6b4f4fe into master Sep 21, 2015
@fhemberger fhemberger deleted the standard-js branch September 21, 2015 15:57
@zeke
Copy link
Contributor

zeke commented Sep 21, 2015

👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants