-
Notifications
You must be signed in to change notification settings - Fork 135
adds deprecation notices for clique #1889
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
Conversation
Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
macfarla
left a comment
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.
technically we only deprecated Clique mining in 25.12.0 - otherwise LGTM
Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
macfarla
left a comment
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.
think for other deprecations like this we don't link to the changelog, I'd keep it simple.
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
e87fda1 to
1a14212
Compare
bgravenorst
left a comment
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.
See suggestion
Signed-off-by: m4sterbunny <harrie.bickle-ext@consensys.net>
Description
Issue(s) fixed
Fixes #1884
Preview