Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Rename squatted crates #2875

Merged
merged 3 commits into from
Aug 15, 2023
Merged

Rename squatted crates #2875

merged 3 commits into from
Aug 15, 2023

Conversation

Morganamilo
Copy link
Contributor

This commit adds the staging- prefix to squatted crates so we can go forward and publish them to crates.io.

Using the staging- prefix is a temp fix until we decide on replacement names. https://forum.parity.io/t/renaming-squated-crates-in-substrate-polkadot-cumulus/1964/6

This commit adds the staging- prefix to squatted crates so we can go
forward and publish them to crates.io.

Using the staging- prefix is a temp fix until we decide on replacement names.
https://forum.parity.io/t/renaming-squated-crates-in-substrate-polkadot-cumulus/1964/6
@Morganamilo Morganamilo added D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. B1-note_worthy Changes should be noted in the release notes T9-release This PR/Issue is related to topics touching the release notes. C1-low PR touches the given topic and has a low impact on builders. T2-API This PR/Issue is related to APIs. and removed T9-release This PR/Issue is related to topics touching the release notes. labels Jul 15, 2023
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

Imo we should just remove the crate. It doesn't bring any benefit

@Morganamilo
Copy link
Contributor Author

A bunch of other crates depend on it. So removing it causes a bunch of compile issues. I'd rather keep this PR to just a simple rename and if some one want's to remove the crate later on they can.

@gilescope
Copy link
Contributor

Shouldn't it be called pallet-parachain-info in line with all the other pallets in substrate? That would remove the squatting problem.

@ggwpez
Copy link
Member

ggwpez commented Jul 24, 2023

Shouldn't it be called pallet-parachain-info in line with all the other pallets in substrate? That would remove the squatting problem.

Many of them are also prefixed with cumulus-pallet-. Going to reserve both (just in case).

This reverts commit 6387eab.

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@Morganamilo
Copy link
Contributor Author

@bkchr can you approve. This PR is to just get things working. Any cleanup or actual changes like removing a crate can be handled after.

@bkchr bkchr merged commit bc1a6ff into master Aug 15, 2023
6 checks passed
@bkchr bkchr deleted the morganamilo/rename-crates branch August 15, 2023 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. T2-API This PR/Issue is related to APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants