Skip to content
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

Bump Cargo.toml version before building #52

Merged
merged 3 commits into from
Jun 23, 2023

Conversation

daladim
Copy link
Contributor

@daladim daladim commented Jun 23, 2023

Before this commit, Cargo.toml was only bumped before publishing, but the binary would already have been built and cached, and any use of e.g. env!("CARGO_PKG_VERSION") would return unexpected values

@daladim daladim force-pushed the bump_cargo_version_before_build branch 2 times, most recently from e12fde8 to 62bf890 Compare June 23, 2023 12:57
Before this commit, Cargo.toml was only bumped before _publishing_, but the binary would already have been built and cached,
and any use of e.g. `env!("CARGO_PKG_VERSION")` would return unexpected values
@daladim daladim force-pushed the bump_cargo_version_before_build branch from d9eca6f to def5b80 Compare June 23, 2023 14:54
@daladim daladim force-pushed the bump_cargo_version_before_build branch from 1ecccae to 1f12823 Compare June 23, 2023 15:15
@daladim daladim force-pushed the bump_cargo_version_before_build branch from 1f12823 to 34eae3d Compare June 23, 2023 15:15
@daladim daladim merged commit 1d034d1 into develop Jun 23, 2023
@daladim daladim deleted the bump_cargo_version_before_build branch June 23, 2023 15:39
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.

None yet

2 participants