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

Remove deprecation notice #212

Closed
ghost opened this issue Jun 25, 2017 · 4 comments
Closed

Remove deprecation notice #212

ghost opened this issue Jun 25, 2017 · 4 comments

Comments

@ghost
Copy link

ghost commented Jun 25, 2017

#209 related.

Need to figure out what's needed to remove the deprecation notice. I've updated the repo title to [CURRENTLY DEPRECATED] to hint at revival. Will dig into this at some point unless some in the org (current or future) gets to it first.

@ghost ghost added the enhancement label Jun 25, 2017
@gtramontina
Copy link
Collaborator

gtramontina commented Jun 27, 2017

To "un-deprecate" it with npm, it is just a matter of setting the deprecation message to an empty string, as per the docs https://docs.npmjs.com/cli/deprecate

To un-deprecate a package, specify an empty string ("") for the message argument.

@ghost
Copy link
Author

ghost commented Jun 27, 2017

@gtramontina thanks. I will look into taking care of this with a new minor as soon as I'm able to get off my current network, which is blocking my 22 and preventing PGP.

@gtramontina
Copy link
Collaborator

Done! 👍 A patch release went out with #218.

@G-Rath
Copy link

G-Rath commented Apr 21, 2020

@gtramontina given that the poster has deleted their github account, and ghooks hasn't seen a new version or activity (aside from dependabot) in a few years, might be time to re-deprecate the package to reduce confusion?

(I stumbled upon an old project using this and had to hunt around a bit to confirm that it's been abandoned).

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

No branches or pull requests

2 participants