Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

XcmContext to buy_weight / refund_weight #7563

Merged
merged 10 commits into from
Aug 1, 2023
Merged

XcmContext to buy_weight / refund_weight #7563

merged 10 commits into from
Aug 1, 2023

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented Jul 31, 2023 •
edited
Loading

@bkontur bkontur changed the base branch from master to pato/xcm-weight-conversion July 31, 2023 12:46
@bkontur bkontur mentioned this pull request Jul 31, 2023
Closed
@bkontur bkontur requested review from a team and chevdor as code owners July 31, 2023 13:35
@bkontur bkontur changed the base branch from pato/xcm-weight-conversion to master July 31, 2023 13:36
@bkontur bkontur added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. T6-XCM This PR/Issue is related to XCM. labels Jul 31, 2023
@bkontur bkontur mentioned this pull request Jul 31, 2023
Merged
@bkontur
Copy link
Contributor Author

bkontur commented Jul 31, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@bkontur
Copy link
Contributor Author

bkontur commented Aug 1, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/3305930

@bkontur
Copy link
Contributor Author

bkontur commented Aug 1, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit c25980c into master Aug 1, 2023
3 checks passed
@paritytech-processbot paritytech-processbot bot deleted the bko-xcm-context-for-everyone branch August 1, 2023 11:04
@muharem muharem added the F3-breaks_API This PR changes public API; next release should be major. label Aug 1, 2023
s0me0ne-unkn0wn pushed a commit that referenced this pull request Aug 15, 2023
@bkontur @PatricioNapoli
@gilescope @s0me0ne-unkn0wn
XcmContext to buy_weight / refund_weight (#7563)
1dfd9ca 
* added XcmContext to WeightTrader trait

* cargo fmt

* make xcm context optional

* make compile

* fix compile

* `XcmContext` to `buy_weight / refund_weight`

---------

Co-authored-by: Patricio Napoli <[email protected]>
Co-authored-by: Giles Cope <[email protected]>
Co-authored-by: parity-processbot <>
@crystalin crystalin mentioned this pull request Oct 20, 2023
Closed
@JesseAbram JesseAbram mentioned this pull request Oct 26, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gilescope gilescope gilescope approved these changes

@KiChjang KiChjang KiChjang approved these changes

@franciscoaguirre franciscoaguirre franciscoaguirre approved these changes

@muharem muharem muharem approved these changes

@vstam1 vstam1 vstam1 approved these changes

@PatricioNapoli PatricioNapoli PatricioNapoli approved these changes

@sergejparity sergejparity sergejparity approved these changes

@chevdor chevdor Awaiting requested review from chevdor

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. F3-breaks_API This PR changes public API; next release should be major. T6-XCM This PR/Issue is related to XCM.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@bkontur @paritytech-cicd-pr @gilescope @KiChjang @franciscoaguirre @muharem @vstam1 @PatricioNapoli @sergejparity