Skip to content

1687-Polkadot v1.1.0 Upgrade follow up for small issues found in final review #1752

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

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

mattheworris
Copy link
Collaborator

@mattheworris mattheworris commented Oct 31, 2023

Goal

The goal of this PR is clean up a few issues from the polkadot v1.1.0 upgrade.

Discussion

  • Remove RUSTSEC-2022-0093 from .cargo-deny.toml.
  • Add w3f to [sources.allow-org] to eliminate some warnings.
  • Remove substrate-wasm-builder from [build-dependencies] in runtime/common/Cargo.toml

Checklist

  • Chain spec updated
  • Custom RPC OR Runtime API added/changed? Updated js/api-augment.
  • Design doc(s) updated
  • Tests added
  • Benchmarks added
  • Weights updated

@github-actions github-actions bot added the metadata-changed Metadata has changed since the latest full release label Nov 1, 2023
@mattheworris mattheworris self-assigned this Nov 1, 2023
@github-actions github-actions bot removed the metadata-changed Metadata has changed since the latest full release label Nov 1, 2023
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #1752 (94490ff) into main (33c1231) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1752      +/-   ##
==========================================
- Coverage   88.07%   88.05%   -0.03%     
==========================================
  Files          50       50              
  Lines        4243     4243              
==========================================
- Hits         3737     3736       -1     
- Misses        506      507       +1     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mattheworris mattheworris changed the title 1687 follow up 1687-Polkadot v1.1.0 Upgrade follow up for small issues found in final review Nov 1, 2023
@mattheworris mattheworris requested a review from a team November 1, 2023 18:34
@mattheworris mattheworris added the chore No feature changes label Nov 1, 2023
@mattheworris mattheworris marked this pull request as ready for review November 1, 2023 18:35
@mattheworris mattheworris requested a review from wilwade as a code owner November 1, 2023 18:35
Copy link
Collaborator

@wilwade wilwade left a comment

Choose a reason for hiding this comment

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

Nice!

@mattheworris mattheworris requested a review from a team November 1, 2023 19:49
Copy link
Collaborator

@enddynayn enddynayn left a comment

Choose a reason for hiding this comment

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

👍 great!

@mattheworris mattheworris merged commit 1831b52 into main Nov 2, 2023
@mattheworris mattheworris deleted the 1687-follow-up branch November 2, 2023 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore No feature changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants