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

Fix CI warnings #7006

Closed
wants to merge 2 commits into from
Closed

Fix CI warnings #7006

wants to merge 2 commits into from

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Apr 4, 2023

Warnings were ignored by the companion check after paritytech/substrate#13798 and are causing CI failures.

Changes:

  • Update to Substrate master
  • Fix warnings

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez ggwpez added A0-please_review Pull request needs code review. 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. labels Apr 4, 2023
@ggwpez ggwpez changed the title Fix warnings Fix CI warnings Apr 4, 2023
@@ -67,7 +67,7 @@ pub mod pallet {
///
/// The new validators will be active from current session + 2.
#[pallet::call_index(0)]
#[pallet::weight(100_000)]
#[pallet::weight({100_000})]
Copy link
Member Author

Choose a reason for hiding this comment

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

This will get a deprecation warning in the future anyway when we deprecate the From<u64> for Weight hacks.

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2641424

@ggwpez
Copy link
Member Author

ggwpez commented Apr 4, 2023

Merged into #7006

@ggwpez ggwpez closed this Apr 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants