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

Rename squatted crates #1241

Merged
merged 4 commits into from
Aug 30, 2023
Merged

Rename squatted crates #1241

merged 4 commits into from
Aug 30, 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

@Morganamilo Morganamilo added T8-polkadot This PR/Issue is related to/affects the Polkadot network. T9-cumulus This PR/Issue is related to cumulus. labels Aug 29, 2023
@paritytech-ci paritytech-ci requested review from a team August 29, 2023 09:00
Copy link
Member

@ordian ordian left a comment

Choose a reason for hiding this comment

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

I assume this is a temporary name?

@ggwpez ggwpez requested a review from KiChjang August 29, 2023 10:50
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3470104

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
@ggwpez
Copy link
Member

ggwpez commented Aug 29, 2023

I assume this is a temporary name?

Semi-temporary. We discussed it in the XCM channel, and it is best effort until either 1.) there is an actual renaming of XCM, 2.) we found a better crate naming scheme to circumvent the squatting or 3.) crates-io actually returns us the crates…

@Morganamilo
Copy link
Contributor Author

sadly 3 is never going to happen due to crate.io policy.

@Morganamilo Morganamilo enabled auto-merge (squash) August 29, 2023 13:40
@Morganamilo Morganamilo requested a review from a team August 29, 2023 14:35
Copy link
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

staging-xcm is fine for a temporary test but I would not suggested this as a more definitive new name.

How about:

  • pxcm
  • subxcm
  • xc-m :)

@ggwpez
Copy link
Member

ggwpez commented Aug 29, 2023

staging-xcm is fine for a temporary test but I would not suggested this as a more definitive new name.

It took very long to even arrive at this compromise, so i dont think it is worth of changing it again now. The name was settled on in the XCM chat, please ask there 😄
image

(full quote was "i guess we'll need to push forward without the proper naming but keep in mind that we'll likely get at least a few of the real names back soon.").

@chevdor
Copy link
Contributor

chevdor commented Aug 29, 2023

@ggwpez oh so the name was already defined, ok.

@KiChjang
Copy link
Contributor

sadly 3 is never going to happen due to crate.io policy.

Untrue, we have contacted the crates.io team before, and we've been assured that there is an upcoming RFC that attempts to change the squatting policy.

Aside from that, I don't care what sort of name we settle into, as long as we have a solution that we can deliver now.

@Morganamilo
Copy link
Contributor Author

I wasn't aware of that. That's good news as their current policy is quite annoying.

@Morganamilo Morganamilo merged commit 31c7947 into master Aug 30, 2023
@Morganamilo Morganamilo deleted the morganamilo/squat branch August 30, 2023 14:57
@ggwpez ggwpez mentioned this pull request Aug 30, 2023
Daanvdplas pushed a commit that referenced this pull request Sep 11, 2023
* Rename squatted crates

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

* Fix test after crate renames

* Update Lockfile
liamaharon added a commit that referenced this pull request Oct 2, 2023
…o CI (#1344)

Makes SPs first class citizens along with the relay chains in the
context of our CI runtime upgrade checks.

## Code changes

- Sets missing current storage version in `uniques` pallet
- Adds multisig V1 migration to run where it was missing
- Removes executed migration whos pre/post hooks were failing from
collectives runtime
- Initializes storage versions for SP pallets added after genesis
- Originally I was going to wait for
#1297 to be merged so
this wouldn't need to be done manually, but it doesn't seem like it'll
be merged any time soon so I've decided to set them manually to unblock
this

## CI changes

- Removed dependency of `westend` runtime upgrades being complete prior
to other ones running. I assume it is supposed to cache the
`try-runtime` build for a performance benefit, but it seems it wasn't
working. Maybe someone from the CI team can look into this or explain
why it needs to be there?

- Adds check-runtime-migration jobs for Parity asset-hubs, bridge-hubs
and contract chains

- Updated VARIABLES to accomodate the `kusama-runtime` package being
renamed to `staging-kusama-runtime` in
#1241

- Added `EXTRA_ARGS` variable to `check-runtime-migration`, and set
`--no-weight-warnings` to the relay chain runtime upgrade checks (relay
chains don't have weight restrictions).
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
…o CI (paritytech#1344)

Makes SPs first class citizens along with the relay chains in the
context of our CI runtime upgrade checks.

## Code changes

- Sets missing current storage version in `uniques` pallet
- Adds multisig V1 migration to run where it was missing
- Removes executed migration whos pre/post hooks were failing from
collectives runtime
- Initializes storage versions for SP pallets added after genesis
- Originally I was going to wait for
paritytech#1297 to be merged so
this wouldn't need to be done manually, but it doesn't seem like it'll
be merged any time soon so I've decided to set them manually to unblock
this

## CI changes

- Removed dependency of `westend` runtime upgrades being complete prior
to other ones running. I assume it is supposed to cache the
`try-runtime` build for a performance benefit, but it seems it wasn't
working. Maybe someone from the CI team can look into this or explain
why it needs to be there?

- Adds check-runtime-migration jobs for Parity asset-hubs, bridge-hubs
and contract chains

- Updated VARIABLES to accomodate the `kusama-runtime` package being
renamed to `staging-kusama-runtime` in
paritytech#1241

- Added `EXTRA_ARGS` variable to `check-runtime-migration`, and set
`--no-weight-warnings` to the relay chain runtime upgrade checks (relay
chains don't have weight restrictions).
bkchr pushed a commit that referenced this pull request Apr 10, 2024
#1241)

* update Rococo version + create relayers fund account

* start finality relay guards when complex relay is started
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. T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants