Skip to content

filter dup tx#2833

Merged
KamiD merged 1 commit intodevfrom
yxq/filter-duptx
Dec 6, 2022
Merged

filter dup tx#2833
KamiD merged 1 commit intodevfrom
yxq/filter-duptx

Conversation

@xzav3r
Copy link
Contributor

@xzav3r xzav3r commented Dec 2, 2022

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

  • Reviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2022

Codecov Report

Merging #2833 (f6b784a) into dev (754b423) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #2833      +/-   ##
==========================================
- Coverage   36.30%   36.30%   -0.01%     
==========================================
  Files         415      415              
  Lines       43814    43814              
==========================================
- Hits        15907    15905       -2     
- Misses      26752    26753       +1     
- Partials     1155     1156       +1     
Impacted Files Coverage Δ
x/wasm/keeper/keeper.go 80.39% <0.00%> (-0.31%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

ilovers
ilovers previously approved these changes Dec 5, 2022
ilovers
ilovers previously approved these changes Dec 5, 2022
@KamiD KamiD merged commit 313891d into dev Dec 6, 2022
@KamiD KamiD deleted the yxq/filter-duptx branch December 6, 2022 02:48
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.

4 participants