Skip to content

Comments

bump version to v0.1.2#3

Merged
SF-Zhou merged 1 commit intomainfrom
dev
Jan 18, 2026
Merged

bump version to v0.1.2#3
SF-Zhou merged 1 commit intomainfrom
dev

Conversation

@SF-Zhou
Copy link
Owner

@SF-Zhou SF-Zhou commented Jan 18, 2026

  1. add dry_run option

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a dry_run option to the blkreader library and CLI tool, allowing users to validate extent mapping and file structure without performing actual I/O operations. The version is bumped from v0.1.1 to v0.1.2.

Changes:

  • Added dry_run field to Options struct with corresponding builder method
  • Implemented dry_run mode in reader logic to skip actual I/O operations
  • Added --dry-run CLI flag to the blkreader binary
  • Updated documentation to describe the new feature

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
Cargo.toml Bumped version from 0.1.1 to 0.1.2
Cargo.lock Updated version lock to 0.1.2
src/options.rs Added dry_run field, default value, and builder method with documentation
src/reader.rs Implemented dry_run logic in fallback_read and DeviceHandle::read_at methods, added test for options builder
src/bin/blkreader.rs Added --dry-run CLI argument and wired it to Options
README.md Documented the --dry-run option and its use cases

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

1. add `dry_run` option
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 6 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SF-Zhou SF-Zhou merged commit 43564a8 into main Jan 18, 2026
7 checks passed
@SF-Zhou SF-Zhou deleted the dev branch January 18, 2026 13:30
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.

1 participant