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

Define common dependencies in Cargo Workspace #1572

Closed
cmackenzie1 opened this issue Jul 28, 2023 · 0 comments · Fixed by #1575
Closed

Define common dependencies in Cargo Workspace #1572

cmackenzie1 opened this issue Jul 28, 2023 · 0 comments · Fixed by #1575
Labels
enhancement New feature or request

Comments

@cmackenzie1
Copy link
Contributor

cmackenzie1 commented Jul 28, 2023

Description

By defining a set of common dependecies in the root Cargo.toml, updates to those crates versions only need to be applied in one location instead of in rust/ and python/ workspaces.

See #1571 as an example of something that would benefit from using [workspace.dependencies]

Use Case

Related Issue(s)

@cmackenzie1 cmackenzie1 added the enhancement New feature or request label Jul 28, 2023
cmackenzie1 added a commit to cmackenzie1-contrib/delta-rs that referenced this issue Jul 31, 2023
This commit moves a common set of dependencies from `rust/` and `python/`
Cargo.toml files in favor of using the `workspace = true` variation.

The goal of this commit if merely organization. Pruning features,
bumping versions are not goals of this commit.

- Closes delta-io#1572
rtyler pushed a commit that referenced this issue Jul 31, 2023
This commit moves a common set of dependencies from `rust/` and `python/`
Cargo.toml files in favor of using the `workspace = true` variation.

The goal of this commit if merely organization. Pruning features,
bumping versions are not goals of this commit.

- Closes #1572
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant