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

chore: merge osmo-v22/v0.47.5 into "main" #501

Merged
merged 15 commits into from
Feb 8, 2024
Merged

Conversation

ValarDragon
Copy link
Member

Please compare commits!

ValarDragon and others added 10 commits January 5, 2024 16:22
* config option to log gRPC queries

* clarify gRPC log comment

* create var
* ci

* lints and remove some unnecessary CI jobs

* update tests go mod

* more lints

* remove denom owners since not implemented in osmosis

* more lints

* fix test values

* implement nolint again

* more test fixes

* add back labeler as v4

* remove tabs and slash

* update json to const

* move file locations

* push

* fix seq num issue

* Update tests/e2e/auth/vesting/suite.go

* add back pwd

* remove sonarcloud

---------

Co-authored-by: Matt, Park <[email protected]>
@czarcas7ic
Copy link
Member

We arent using this branch anymore so i dont think this needs to be done

@@ -6,5 +6,5 @@
)

func EndBlocker(ctx sdk.Context, k keeper.Keeper) {
k.RemoveExpiredAllowances(ctx)
k.RemoveExpiredAllowances(ctx, 200)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods Warning

path flow from Begin/EndBlock to a panic call
path flow from Begin/EndBlock to a panic call
path flow from Begin/EndBlock to a panic call
path flow from Begin/EndBlock to a panic call
path flow from Begin/EndBlock to a panic call
* add testnetify logic

* clarify explanation

* handle errors

* remove testnet app creator

* simplify some logic

* remove redundancy

* lints

* gci imports

* add set store loader method

* trigger upgrade flag

* add extra comments to the CLI command
Height: state.LastBlockHeight,
Round: 0,
BlockID: state.LastBlockID,
Timestamp: time.Now(),

Check warning

Code scanning / CodeQL

Calling the system time Warning

Calling the system time may be a possible source of non-determinism
@czarcas7ic czarcas7ic changed the title Osmo v22/v0.47.5 chore: merge osmo-v22/v0.47.5 into "main" Feb 8, 2024
@czarcas7ic czarcas7ic merged commit b794dab into osmo/v0.47.5 Feb 8, 2024
22 of 24 checks passed
czarcas7ic added a commit that referenced this pull request Feb 8, 2024
czarcas7ic added a commit that referenced this pull request Feb 8, 2024
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.

3 participants