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
chore: format and standardize all markdowns files #2726
Conversation
according to the github flavor (https://github.github.com/gfm/)
- `*_hex`: consensus encoded types represented as hex strings | ||
- `*_ser`: consensus encoded types represented as binary data | ||
- `*_bincode`: types serialized with `serde` as bincode |
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.
Opinionated: this should be highlighted as code
We consensus deserialize, `serde` serialize, then check against the expected data | ||
to verify no `serde` regressions have been introduced. |
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.
We already highlight "serde" as inline code, so this is standardizing it.
Pull Request Test Coverage Report for Build 8858884788Details
💛 - Coveralls |
|
||
# Fuzzing with weak cryptography | ||
## Fuzzing with weak cryptography |
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.
A markdown file should have only one H1.
```bash | ||
cargo update -p serde --precise 1.0.156 | ||
cargo update -p regex --precise 1.7.3 | ||
cargo update -p chrono --precise 0.4.24 | ||
``` |
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.
Opinionated: this should be a fenced block for nice syntax highlighting
|
||
The `std::io` module is not exposed in `no-std` Rust so building `no-std` applications which require | ||
reading and writing objects via standard traits is not generally possible. Thus, this library exists | ||
to export a minmal version of `std::io`'s traits which we use in `rust-bitcoin` so that we can | ||
to export a minimal version of `std::io`'s traits which we use in `rust-bitcoin` so that we can |
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.
typo
@@ -23,7 +23,7 @@ Rust Logo is licensed under CC-BY, which allows reuse and modifications for any | |||
|
|||
## Acknowledgements | |||
|
|||
Acknowledgement for the runners up in this PR: https://github.com/rust-bitcoin/rust-bitcoin/pull/891#issuecomment-1074476858 | |||
Acknowledgement for the runners up in this PR: <https://github.com/rust-bitcoin/rust-bitcoin/pull/891#issuecomment-1074476858> |
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.
Automatic formatted by my linter.
We can revert if necessary.
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.
ACK b355740 thanks for going through all this!
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.
Sweeet, thanks man!
ACK b355740
according to the github flavor
(https://github.github.com/gfm/)