-
Notifications
You must be signed in to change notification settings - Fork 104
chore: Prepare cargo-wdk for release #560
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
Conversation
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.
Pull Request Overview
This PR prepares cargo-wdk for its initial crates.io release by updating documentation, fixing minor issues, and updating package metadata to version 0.1.0.
- Version bumped from 0.0.1 to 0.1.0 and package set to be publishable
- README completely rewritten with improved structure and clearer usage examples
- Comment capitalization fixed for consistency
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| crates/cargo-wdk/Cargo.toml | Version bumped to 0.1.0, publish flag set to true, tests excluded from package |
| Cargo.lock | Version updated to match package change |
| crates/cargo-wdk/README.md | Complete rewrite with better structure, clearer examples, and updated installation instructions |
| crates/cargo-wdk/src/cli.rs | Fixed comment capitalization for consistency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
krishnakumar4a4
left a 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.
lgtm
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.
Pull Request Overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
svasista-ms
left a 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.
LGTM
This PR makes the following changes to
cargo-wdkin preparation to publish it to crates.io:0.0.1to0.1.0publishtotrueexcludestatement to exclude thecrates/cargo-wdk/testsdirectory from the publicationREADME.mdto suggest installing from crates.io instead of GitHubREADME.mdin general to make it clearer and better structured.README changes
Below are before and after screenshots of
README.mdto give a sense of what the overhaul looks like.Before
After