-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 ethpm #3261
Remove ethpm #3261
Conversation
abf72f0
to
49b84c6
Compare
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.
lgtm!
Does this removal require any user docs for migrating away from ethpm?
i don't believe there are any current users (it's been borked for ~18 mos), but good reminder to at least put a note in the migration guide. Done! |
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.
-13k... love that. lgtm 👍🏼
What was wrong?
ethpm
was deprecated, now being fully removed.Related to Issue #2854
Todo:
Cute Animal Picture