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

Bump clap to 4.2.5 #14061

Merged
merged 2 commits into from May 2, 2023
Merged

Bump clap to 4.2.5 #14061

merged 2 commits into from May 2, 2023

Conversation

altaua
Copy link
Contributor

@altaua altaua commented May 2, 2023

We need a clap version incorporating clap-rs/clap#4739 to work around a Rust 1.69.0 clippy bug.

As far as I can see in the clap release notes, there have been no relevant changes between clap 4.0.9 and 4.2.5 that'd require any action on our side.

Prerequisite of https://github.com/paritytech/ci_cd/issues/789

polkadot companion: paritytech/polkadot#7160
cumulus companion: paritytech/cumulus#2505

@altaua altaua added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. E2-dependencies Pull requests that update a dependency file. labels May 2, 2023
@altaua altaua self-assigned this May 2, 2023
@altaua altaua requested review from bkchr and skunert May 2, 2023 13:55
Copy link

@coderobe coderobe left a comment

Choose a reason for hiding this comment

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

i recall a previous clap bump subtly breaking behaviour that was being relied on - this should probably be smoke-tested locally

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.

Please update the Cargo.toml files here in Substrate.

@altaua
Copy link
Contributor Author

altaua commented May 2, 2023

Please update the Cargo.toml files here in Substrate.

Can do, but why? I'm just updating clap because it's useful for CI, but the new version isn't strictly required by the substrate code.

@altaua
Copy link
Contributor Author

altaua commented May 2, 2023

i recall a previous clap bump subtly breaking behaviour that was being relied on - this should probably be smoke-tested locally

Our testsuites run fine and the changelog sounds reasonable; do you have anything in particular you'd like tested beyond that?

@bkchr
Copy link
Member

bkchr commented May 2, 2023

Please update the Cargo.toml files here in Substrate.

Can do, but why? I'm just updating clap because it's useful for CI, but the new version isn't strictly required by the substrate code.

Because otherwise the changes here are not required. As you already have done it downstream.

@bkchr
Copy link
Member

bkchr commented May 2, 2023

Others will also run into the issues with the latest rust version, but if we bump here the versions in the Cargo.toml they would get these changes "for free" in the downstream projects.

@altaua altaua added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label May 2, 2023
@ggwpez
Copy link
Member

ggwpez commented May 2, 2023

bot rebase

CI was red.

@paritytech-processbot
Copy link

Rebased

@altaua
Copy link
Contributor Author

altaua commented May 2, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 7212e91 into master May 2, 2023
45 checks passed
@paritytech-processbot paritytech-processbot bot deleted the mira/clap branch May 2, 2023 17:54
gpestana pushed a commit that referenced this pull request May 4, 2023
Co-authored-by: parity-processbot <>
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
Co-authored-by: parity-processbot <>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit E2-dependencies Pull requests that update a dependency file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants