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

chore: polkadot v1.10.0 #171

Merged
merged 4 commits into from
May 20, 2024
Merged

chore: polkadot v1.10.0 #171

merged 4 commits into from
May 20, 2024

Conversation

Daanvdplas
Copy link
Collaborator

The mock.rs file had a change coming from the v1.10 upgrade. The Cargo.toml files had both one as well (codec). Right now I changed everything to workspace dependencies which would be ideal. However, when someone creates a new project the pallets/ folder is not yet added to the workspace in the root Cargo.toml file. Putting it here to get your opinion and see if it can easily be resolved.

@Daanvdplas Daanvdplas requested a review from AlexD10S May 15, 2024 06:56
@Daanvdplas
Copy link
Collaborator Author

Daanvdplas commented May 18, 2024

I want to update the links in the pallet template because they are outdated. Will finish as soon as I know what the best place is.

@Daanvdplas Daanvdplas marked this pull request as ready for review May 20, 2024 06:47
@Daanvdplas Daanvdplas force-pushed the daan/chore-polkadot_v1.10.0 branch from e4608bc to 3acdcfd Compare May 20, 2024 07:08
Copy link

codecov bot commented May 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.28%. Comparing base (681e6ed) to head (3acdcfd).
Report is 6 commits behind head on main.

@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
+ Coverage   48.43%   50.28%   +1.85%     
==========================================
  Files          32       32              
  Lines        2707     2828     +121     
  Branches     2707     2828     +121     
==========================================
+ Hits         1311     1422     +111     
+ Misses       1194     1183      -11     
- Partials      202      223      +21     

see 8 files with indirect coverage changes

@Daanvdplas Daanvdplas requested a review from AlexD10S May 20, 2024 08:32
@Daanvdplas Daanvdplas merged commit 047bee3 into main May 20, 2024
14 checks passed
@Daanvdplas Daanvdplas deleted the daan/chore-polkadot_v1.10.0 branch May 20, 2024 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants