Skip to content

Fixes #2952 - Fixes husky pre-commit deprecation#3102

Merged
miketaylr merged 1 commit intowebcompat:masterfrom
magbello157:2952/1
Dec 4, 2019
Merged

Fixes #2952 - Fixes husky pre-commit deprecation#3102
miketaylr merged 1 commit intowebcompat:masterfrom
magbello157:2952/1

Conversation

@magbello157
Copy link
Copy Markdown

@magbello157 magbello157 commented Nov 30, 2019

This PR fixes issue #2952

Proposed PR background

Removes the "precommit" key from "scripts" in package.json since that is deprecated and moved to husky.hooks. This eliminates the warning message seen when one makes a commit.

@karlcow karlcow requested a review from miketaylr November 30, 2019 13:28
@miketaylr
Copy link
Copy Markdown
Member

Thanks @meetmuhd!

(Just got back from time off, sorry for the delay)

@miketaylr miketaylr changed the title Issue #2952 - Fixes husky pre-commit deprecation Fixes #2952 - Fixes husky pre-commit deprecation Dec 4, 2019
@miketaylr
Copy link
Copy Markdown
Member

@meetmuhd one small nit, when opening a PR please give a title of the following format:

Fixes #nnnn - summary here. That way when we merge, it will close the issue.

@magbello157
Copy link
Copy Markdown
Author

@meetmuhd one small nit, when opening a PR please give a title of the following format:

Fixes #nnnn - summary here. That way when we merge, it will close the issue.

Oh my bad! I think I mistakenly put what was meant to be the title into a comment instead. My apologies please. And thank you so much for the merge and feedback! @miketaylr

@magbello157 magbello157 deleted the 2952/1 branch December 4, 2019 21:12
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.

2 participants