chore: modernize project to Rust 2024 and update configurations#1
Open
fenris02 wants to merge 4 commits intofcami:mainfrom
Open
chore: modernize project to Rust 2024 and update configurations#1fenris02 wants to merge 4 commits intofcami:mainfrom
fenris02 wants to merge 4 commits intofcami:mainfrom
Conversation
- Updated the project to the Rust 2024 edition and set MSRV to 1.91. - Added templates and configurations for linting, formatting, and licensing: - Added `.clippy.toml`, `.rustfmt.toml`, `.dprint.json`, `.prettierrc`, and `.editorconfig`. - Added `.licensesnip` for license header management and `.typos.toml` for spell checking. - Applied automated maintenance and code quality fixes: - Ran `cargo fix` and `cargo clippy --fix`. - Upgraded dependencies using `cargo update` and `cargo upgrade`. - Formatted the codebase with `cargo fmt` and `dprint fmt`. - Refined project lints in `Cargo.toml`: - Set `unsafe_code` to `warn` to allow `cargo audit` and `cargo clippy` to run with only warnings about unsafe usage. - Enabled pedantic clippy lints. - Documentation updates: - Added `docs/analyze_this_project,_what_problems_can.md` outlining architectural analysis and future improvements. - Generated `docs/cargo-license.txt` for dependency license tracking.
Owner
|
hi @fenris02 thank you for the contribution! We currently need to be able to build with older Rust toolchain versions (down to what RHEL 8 provides). I will drop RHEL 8 soon, but that leaves RHEL 9. Can you please check if you can build your branch using the UBI steps in the readme? |
added 3 commits
March 26, 2026 13:57
…ration - Replaced `cargo test` with `cargo nextest run` across all testing commands. - Added `.nextest.toml` configuration for customizable and efficient test execution.
…mplementations - Added `from_bytes` methods for `McHeader`, `McRec`, and related types in `src/types.rs`. - Replaced all `std::ptr::read_unaligned` usages with corresponding `from_bytes` calls. - Improved error handling for parsing with detailed length checks and `Option`/`Result` usage. - Added unit test support for `McPwdData` serialization and deserialization.
…cache file handling - Replaced `memmap2::Mmap` with `Vec<u8>` for safer and more portable memory management. - Updated `CacheFile` and related methods to work with in-memory buffers. - Strengthened linter configuration by forbidding `unsafe_code` instead of warning.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
.clippy.toml,.rustfmt.toml,.dprint.json,.prettierrc, and.editorconfig..licensesnipfor license header management and.typos.tomlfor spell checking.cargo fixandcargo clippy --fix.cargo updateandcargo upgrade.cargo fmtanddprint fmt.Cargo.toml:unsafe_codetowarnto allowcargo auditandcargo clippyto run with only warnings about unsafe usage.docs/analyze_this_project,_what_problems_can.mdoutlining architectural analysis and future improvements.docs/cargo-license.txtfor dependency license tracking.