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

workspace package metadata #1036

Merged
merged 9 commits into from
May 31, 2024
Merged

Conversation

gibbz00
Copy link
Contributor

@gibbz00 gibbz00 commented Apr 21, 2024

Depends on #1033 (merged into master and rebased here)

Copy link
Collaborator

@caspermeijn caspermeijn left a comment

Choose a reason for hiding this comment

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

Good idea. Please rebase as #1033 is merged.

prost-build/Cargo.toml Show resolved Hide resolved
prost-build/Cargo.toml Outdated Show resolved Hide resolved
@@ -6,7 +6,7 @@ license.workspace = true
repository.workspace = true
readme = "README.md"
description.workspace = true
edition = "2021"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we should only centralize values that are the same for all crates. It can be confusing when the edition is set at the workspace level, but different for other crates.

So we should update the edition of the other crates before doing this commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Separate PR: #1039

Copy link
Collaborator

Choose a reason for hiding this comment

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

Now that the private projects have the same edition, then can also use the workspace.edition.

@caspermeijn
Copy link
Collaborator

Now that the private projects have the same edition, then can also use the workspace.edition.

@gibbz00 Could you add edition.workspace = true to the all crates?

Copy link
Collaborator

@caspermeijn caspermeijn left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution

@caspermeijn caspermeijn added this pull request to the merge queue May 31, 2024
Merged via the queue into tokio-rs:master with commit e7049d3 May 31, 2024
15 checks passed
caspermeijn added a commit to caspermeijn/prost that referenced this pull request Jul 5, 2024
The prepare release script got out of sync with the actual code.
- Automatically update the minor version in the readme
- Cargo.toml version was moved in tokio-rs#1036
github-merge-queue bot pushed a commit that referenced this pull request Jul 8, 2024
The prepare release script got out of sync with the actual code.
- Automatically update the minor version in the readme
- Cargo.toml version was moved in #1036
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