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

Use Safe Math in XCM Builder #3249

Merged
1 commit merged into from
Jun 16, 2021
Merged

Use Safe Math in XCM Builder #3249

1 commit merged into from
Jun 16, 2021

Conversation

shawntabrizi
Copy link
Member

@shawntabrizi shawntabrizi added B0-silent Changes should not be mentioned in any release notes C3-medium PR touches the given topic and has a medium impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. labels Jun 14, 2021
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jun 14, 2021
@shawntabrizi shawntabrizi added B7-runtimenoteworthy and removed B0-silent Changes should not be mentioned in any release notes labels Jun 15, 2021
@stze stze added D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited. and removed D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited. labels Jun 16, 2021
apopiak
apopiak approved these changes Jun 16, 2021
View reviewed changes
Copy link
Contributor

@apopiak apopiak left a comment

Choose a reason for hiding this comment

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

LGTM

@drahnr
Copy link
Contributor

drahnr commented Jun 16, 2021

bot merge

@ghost
Copy link

ghost commented Jun 16, 2021

Trying merge.

@ghost ghost merged commit 5161ae5 into master Jun 16, 2021
@ghost ghost deleted the shawntabrizi-xcm-safe branch June 16, 2021 10:19
s3krit pushed a commit that referenced this pull request Jun 17, 2021
@shawntabrizi @s3krit
use safe math (#3249)
ea7f0d1
ordian added a commit that referenced this pull request Jun 17, 2021
@ordian
Merge branch 'master' into ao-improved-gossip-topology
e59d267 
* master:
  Companion #9019 (max rpc payload override) (#3276)
  Implementers' Guide: Chain Selection (#3262)
  CLI: Add missing feature checking and check if someone passes a file (#3283)
  Export 'TakeRevenue' trait. (#3278)
  Add XCM Decode Limit (#3273)
  Allow Council to Use Scheduler (#3237)
  fix xcm pallet origin (#3272)
  extract determine_new_blocks into a separate utility (#3261)
  Approval checking unit tests (#3252)
  bridges: update finality-grandpa to 0.14.1 (#3266)
  malus - mockable overseer mvp (#3224)
  use safe math (#3249)
  Companion for #8920 (Control Staking) (#3260)
  Companion for #8949 (#3216)
@github-actions github-actions bot mentioned this pull request Jul 8, 2021
Closed
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@drahnr drahnr drahnr approved these changes

@0x7CFE 0x7CFE 0x7CFE approved these changes

@apopiak apopiak apopiak approved these changes

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. C3-medium PR touches the given topic and has a medium impact on builders. D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@shawntabrizi @drahnr @0x7CFE @apopiak @stze