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

Umbrella crate: exclude chain-specific crates #5173

Merged
merged 7 commits into from
Aug 7, 2024
Merged

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Jul 29, 2024

Uses custom metadata to exclude chain-specific crates.
The only concern is that devs who want to use chain-specific crates, still need to select matching versions numbers. Could possibly be addresses with chain-specific umbrella crates, but currently it should be possible to use psvm.

@paritytech-review-bot paritytech-review-bot bot requested a review from a team July 29, 2024 10:11
@ggwpez ggwpez added the T8-polkadot This PR/Issue is related to/affects the Polkadot network. label Jul 29, 2024
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez enabled auto-merge July 30, 2024 00:04
@ggwpez ggwpez added this pull request to the merge queue Aug 7, 2024
Merged via the queue into master with commit 0fb6e3c Aug 7, 2024
141 of 161 checks passed
@ggwpez ggwpez deleted the oty-trunc-umbrella branch August 7, 2024 15:03
ordian added a commit that referenced this pull request Aug 7, 2024
* master:
  Umbrella crate: exclude chain-specific crates (#5173)
  Bring reference_hardware.json inline with machine used for weights (#5196)
  Snowbridge on Westend (#5074)
  Run semver check even when no prdoc (#5189)
  Export more from sc-service (#5250)
  Update the wishlist leaderboard script to handle PRs (#5256)
dharjeezy pushed a commit to dharjeezy/polkadot-sdk that referenced this pull request Aug 28, 2024
Uses custom metadata to exclude chain-specific crates.  
The only concern is that devs who want to use chain-specific crates,
still need to select matching versions numbers. Could possibly be
addresses with chain-specific umbrella crates, but currently it should
be possible to use [psvm](https://github.com/paritytech/psvm).

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T8-polkadot This PR/Issue is related to/affects the Polkadot network.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants