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

Make base fee configurable from storage. #1060

Merged
merged 4 commits into from
Dec 16, 2023

Conversation

alistair-singh
Copy link
Contributor

@alistair-singh alistair-singh commented Dec 16, 2023

Polkadot-sdk: Snowfork/polkadot-sdk#68

@@ -5,7 +5,7 @@

mod test_cases;

use asset_hub_rococo_runtime::xcm_config::bridging::to_ethereum::BridgeHubEthereumBaseFeeInROC;
use asset_hub_rococo_runtime::xcm_config::bridging::to_ethereum::DefaultBridgeHubEthereumBaseFeeInROC;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using the const version in the tests as the storage version causes it to panic due to there is no underlying storage provider.

@alistair-singh alistair-singh force-pushed the alistair/make-base-fee-storage branch from b60864a to 6ee36eb Compare December 16, 2023 13:18
Copy link

codecov bot commented Dec 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6f14f70) 81.30% compared to head (8c8360b) 81.30%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1060   +/-   ##
=======================================
  Coverage   81.30%   81.30%           
=======================================
  Files          54       54           
  Lines        2241     2241           
  Branches       71       71           
=======================================
  Hits         1822     1822           
  Misses        402      402           
  Partials       17       17           
Flag Coverage Δ
rust 81.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alistair-singh alistair-singh merged commit 09a9253 into main Dec 16, 2023
7 checks passed
@alistair-singh alistair-singh deleted the alistair/make-base-fee-storage branch December 16, 2023 13:44
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.

1 participant