-
Notifications
You must be signed in to change notification settings - Fork 18
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(upgrade): v1.4.0 to v1.5.0 #2034
chore(upgrade): v1.4.0 to v1.5.0 #2034
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
runtime/common/src/fee.rs
Outdated
fn check_weight() { | ||
let p = CENTS / 10; | ||
let q = Balance::from(ExtrinsicBaseWeight::get().ref_time()); | ||
// #[test] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this test commented out? Should it be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question. I am working on getting the new values. The improvements in weights are causing this test to fail. I commented out so that I can run the weights.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mattheworris I wanted to follow up regarding the test we implemented to monitor improvements or regressions in the extrinsic weights. After reviewing it, I have updated the test to reflect the new range of weights based on the improvements observed.
We are not planning to make any changes to our fees at this time. However, please feel free to share your opinion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Read through changes
- Non-blocking comment
lgtm
@@ -40,7 +40,7 @@ impl<Block: BlockT> HeaderBackend<Block> for TestApi { | |||
|
|||
fn status( | |||
&self, | |||
_id: <Block as sp_api::BlockT>::Hash, | |||
_id: Block::Hash, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
f147b27
to
43249ad
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!!
49b8afb
to
c289442
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
c289442
to
30d9cda
Compare
- Upgrade Polkadot-sdk to v.1.5.0. - Update weights to reflect the new version. - Modify extrinsic weight test due to a 25% improvement in weights. For more details, please refer to: [Release Notes](https://github.com/paritytech/polkadot-sdk/releases/tag/polkadot-v1.5.0) [Release Analysis](https://forum.polkadot.network/t/polkadot-release-analysis-v1-5-0/5358) issue-1813
30d9cda
to
2333bf3
Compare
For more details, please refer to:
Release Notes
Release Analysis
#1813