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
move all_zeroes
& from_byte_array
to inherents & make them const.
#2641
base: master
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 8583563910Details
💛 - Coveralls |
1d6373d looks good to me other than the grammar thing. Also, the CI failure is minor but real -- please remove the now-unused |
https://github.com/rust-bitcoin/rust-bitcoin/actions/runs/8490482071/job/23261584927?pr=2641 Can you please tell how to solve this error. |
Recently noticed -> our code is not formatted according to rust fmt tool (cargo fmt). |
Thanks for the PR, this is not a refactor, its an API breaking change because it removes functions from a public trait. I always link contributors to https://chris.beams.io/posts/git-commit/ (also mentioned in CONTRIBUTING.md). If a dev describes why we should have a change, and what the change is then it clarifies a bunch of things, including that:
I don't think you would have written "refactor" after writing a why/what PR description for the changes in this PR :) |
Converting to draft because we are in a feature freeze right now, this needs to wait until |
Well ,I agree that it is more often a api breaking change but while writting conventional commit ->I could'nt find more relevant tag than 'refactor' ->
👍 |
"Refactor" is a well defined term:
ref: https://en.wikipedia.org/wiki/Code_refactoring. We don't enforce any particular format of git brief messages, so there is no requirement to put any sort of tag at the start of the message. Just something simple like "Use inherent functions instead of trait methods" would be fine. Thanks |
Made these functions const fn & move them from the `Hash` trait to `hash_type` & `hash_newtype` macros fix rust-bitcoin#2377
all_zeroes
& from_byte_array
to inherents & make them const.all_zeroes
& from_byte_array
to inherents & make them const.
Feature freeze is over man, you are free to push this up if you want to. |
Made these functions const fn & move them from the
Hash
trait tohash_type
&hash_newtype
macrosfix #2377