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

Backport XCM fixes #3194

Closed
wants to merge 5 commits into from

Commits on Feb 3, 2024

  1. Configuration menu
    Copy the full SHA
    8ca8ae7 View commit details
    Browse the repository at this point in the history
  2. [ci] Add -D warnings for cargo-check-each-crate job to fail on wa…

    …rnings (#2670)
    
    This PR turns on `-D warnings` for `cargo-check-each-crate job` job to
    fail on warnings e.g. like this:
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673130
    
    Before this PR, there was a warning and `cargo-check-each-crate` job did
    not fail:
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4641444
    ```
    warning: unused import: `ToTokens`
      --> substrate/primitives/api/proc-macro/src/utils.rs:22:34
       |
    22 | use quote::{format_ident, quote, ToTokens};
       |                                  ^^^^^^^^
       |
       = note: `#[warn(unused_imports)]` on by default
    warning: `sp-api-proc-macro` (lib) generated 1 warning (run `cargo fix --lib -p sp-api-proc-macro` to apply 1 suggestion)
    ```
    
    Fixes on the way:
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4641444
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673265
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673410
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673681
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673836
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4673941
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4674256
    https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/4679328
    
    - [ ] why does this check triggers only `cargo check --locked`?
    `--all-features` or `--all-targets` are not needed? Or aren't they
    avoided intentionally?
    
    ---------
    
    Co-authored-by: command-bot <>
    bkontur authored and franciscoaguirre committed Feb 3, 2024
    Configuration menu
    Copy the full SHA
    e043aac View commit details
    Browse the repository at this point in the history
  3. Fix clippy issues

    franciscoaguirre committed Feb 3, 2024
    Configuration menu
    Copy the full SHA
    9c951fe View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    6a21fe6 View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2024

  1. [ci] Update rust in ci image (1.75 and 2024-01-22) (#3016)

    cc paritytech/ci_cd#926
    
    ---------
    
    Signed-off-by: Oliver Tale-Yazdi <[email protected]>
    Co-authored-by: command-bot <>
    Co-authored-by: Oliver Tale-Yazdi <[email protected]>
    Co-authored-by: Bastian Köcher <[email protected]>
    Co-authored-by: Liam Aharon <[email protected]>
    Co-authored-by: Dónal Murray <[email protected]>
    Co-authored-by: Vladimir Istyufeev <[email protected]>
    6 people authored and franciscoaguirre committed Feb 5, 2024
    Configuration menu
    Copy the full SHA
    a455c80 View commit details
    Browse the repository at this point in the history