-
Notifications
You must be signed in to change notification settings - Fork 67
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
(not for merge, discussion comments) enum discriminant de- and serialization behavior #148
Conversation
@iho Please, add tests to cover #148 (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.
- Document
use_discriminant
attribute in the README: https://github.com/near/borsh-rs#features
650bab0
to
2c2cb64
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.
Overall, looks good to me, but I delegate the final review and merge this PR to @dj8yfo
@dj8yfo done. Please look again |
reopening to check resolution of comments, not for merge |
closing as all comments have either been resolved or moved to #183 |
… behavior #148 (#183) * WIP * Update insta snapshots * clippy * clippy * Add documentation * remove debug prints; add documentation * fix doc * remove debug print * Apply suggestions from code review Co-authored-by: dj8yf0μl <26653921+dj8yfo@users.noreply.github.com> * Add test; bump Rust version in CI; fix typos * refactoring * Apply suggestion for check_item_attributes * Add docs for * remove comment; add documentation * clippy * replace panic in macros to explicit error * Apply suggestions from code review Co-authored-by: dj8yf0μl <26653921+dj8yfo@users.noreply.github.com> * fix test * Fix comments; Add suggested tests * Add documentation; Bump Rust compiler version * Apply suggestions from code review Co-authored-by: dj8yf0μl <26653921+dj8yfo@users.noreply.github.com> * Fix tests * Fix test; clippy * Apply suggestions from code review Co-authored-by: dj8yf0μl <26653921+dj8yfo@users.noreply.github.com> * Move enum related tests to separate file * fix doc CI * docs: Update "Enum with explicit discriminant" README.md * fix(ci): conditional downgrade of time for older toolchain * ci: fixed the condition on when to downgrade `time` crate in CI workflow * Fix English in doc * chore: revert unneeded imports reordering * chore: duplicate `if let Err(...)` instead of `match` --------- Co-authored-by: dj8yf0μl <26653921+dj8yfo@users.noreply.github.com> Co-authored-by: Vlad Frolov <frolvlad@gmail.com> Co-authored-by: dj8yf0μl <sffhox@gmail.com>
Addressing #138 (comment)
Resolves #147