Skip to content

docs: add multi-file patch guidance#78

Merged
bmwill merged 1 commit into
bmwill:masterfrom
weihanglo:readme
Apr 21, 2026
Merged

docs: add multi-file patch guidance#78
bmwill merged 1 commit into
bmwill:masterfrom
weihanglo:readme

Conversation

@weihanglo

Copy link
Copy Markdown
Contributor

cc #68.

Comment thread src/lib.rs
//! This crate is `no_std` by default.
//! Enable [Cargo features] as needed:
//!
//! - `std` for writer-based formatting and `std::error::Error` impls

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weihanglo question for you on this one. We could theoretically bump the msrv up a bit more to:

  • get Error trait from core
  • update hashbrown to latest version
  • fix a few TODOs you have to use chunk apis
  • update to 2024 edition

Does cargo have a requirement for using a lower msrv? otherwise we can likely bump ours to fix the above. I've already checked the other biggest users of diffy on crates.io and they don't seem to be relying on a low msrv

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cargo uses latest stable, so should be fairly fine with any MSRV bumps

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll whip up a PR and put it up to see if it would be worth doing

@bmwill bmwill merged commit a76e55a into bmwill:master Apr 21, 2026
20 checks passed
@weihanglo weihanglo deleted the readme branch April 21, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants