Skip to content

chore: Use cargo workspaces#260

Merged
mhovd merged 3 commits into
mainfrom
publish-dsl
May 14, 2026
Merged

chore: Use cargo workspaces#260
mhovd merged 3 commits into
mainfrom
publish-dsl

Conversation

@mhovd
Copy link
Copy Markdown
Collaborator

@mhovd mhovd commented May 14, 2026

In order to better handle the monorepo with multiple packages

Copilot AI review requested due to automatic review settings May 14, 2026 11:44
Copy link
Copy Markdown
Contributor

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 updates the repository to use a Cargo workspace setup so the monorepo’s crates can share package metadata/dependency versions and be released together more cleanly.

Changes:

  • Introduces release-plz.toml workspace configuration to publish workspace crates in lockstep.
  • Moves common crate metadata (version/edition/authors/license/repository) into [workspace.package] and centralizes shared deps in [workspace.dependencies].
  • Updates member crates to inherit metadata via *.workspace = true and switches internal deps to workspace = true.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
release-plz.toml Adds release-plz workspace configuration for lockstep releases across all workspace crates.
Cargo.toml Converts root to a workspace-centric layout with shared package metadata and workspace dependencies.
pharmsol-dsl/Cargo.toml Inherits workspace metadata/deps and enables publishing.
pharmsol-macros/Cargo.toml Inherits workspace metadata and uses workspace dependency on pharmsol-dsl.

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

Comment thread Cargo.toml
Comment on lines +13 to +14
pharmsol-dsl = { path = "pharmsol-dsl", version = "0.27.0" }
pharmsol-macros = { path = "pharmsol-macros", version = "0.27.0" }
@github-actions
Copy link
Copy Markdown
Contributor

🐰 Bencher Report

Branchpublish-dsl
Testbedmhovd-pgx

⚠️ WARNING: Truncated view!

The full continuous benchmarking report exceeds the maximum length allowed on this platform.

🐰 View full continuous benchmarking report in Bencher

@mhovd mhovd merged commit 8df3664 into main May 14, 2026
8 checks passed
This was referenced May 14, 2026
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.

3 participants