Skip to content

chore(uibc): update quota -> outflows#1865

Merged
robert-zaremba merged 5 commits intomainfrom
robert/outflows
Feb 23, 2023
Merged

chore(uibc): update quota -> outflows#1865
robert-zaremba merged 5 commits intomainfrom
robert/outflows

Conversation

@robert-zaremba
Copy link
Copy Markdown
Contributor

Description

@robert-zaremba robert-zaremba requested a review from a team as a code owner February 22, 2023 16:10
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 22, 2023

Codecov Report

Merging #1865 (6001dee) into main (ce9ca4a) will increase coverage by 0.00%.
The diff coverage is 46.77%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1865   +/-   ##
=======================================
  Coverage   56.07%   56.07%           
=======================================
  Files          93       93           
  Lines        8803     8802    -1     
=======================================
  Hits         4936     4936           
+ Misses       3502     3501    -1     
  Partials      365      365           
Impacted Files Coverage Δ
x/uibc/keys.go 0.00% <0.00%> (ø)
x/uibc/query.pb.gw.go 0.00% <0.00%> (ø)
x/uibc/genesis.go 55.55% <42.85%> (ø)
x/uibc/quota/keeper/quota.go 38.88% <56.25%> (+0.23%) ⬆️
x/uibc/quota/keeper/grpc_query.go 71.42% <66.66%> (ø)
x/uibc/quota/keeper/genesis.go 100.00% <100.00%> (ø)

if quotaExpires == nil || quotaExpires.Before(ctx.BlockTime()) {
util.Panic(keeper.ResetQuota(ctx))
util.Panic(keeper.ResetAllQuotas(ctx))
}

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if quotaExpires == nil || quotaExpires.Before(ctx.BlockTime()) {
util.Panic(keeper.ResetQuota(ctx))
util.Panic(keeper.ResetAllQuotas(ctx))
}

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
@robert-zaremba
Copy link
Copy Markdown
Contributor Author

@gsk967 you should not block a PR if your suggestions are only about updating code comments.

Copy link
Copy Markdown
Collaborator

@gsk967 gsk967 left a comment

Choose a reason for hiding this comment

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

LGTM

@robert-zaremba robert-zaremba added this pull request to the merge queue Feb 23, 2023
Merged via the queue into main with commit 3dee6cc Feb 23, 2023
@robert-zaremba robert-zaremba deleted the robert/outflows branch February 23, 2023 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants