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

feat: bump cosmos-sdk to v0.47.1 #1011

Merged
merged 18 commits into from
Apr 26, 2023
Merged

feat: bump cosmos-sdk to v0.47.1 #1011

merged 18 commits into from
Apr 26, 2023

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Apr 17, 2023

Closes #. in case of a bug fix, this should point to a bug or any other related issue(s)

What does this PR do?

How to test?

Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Include tests
  • Respect code style and lint
  • Update documentation (*.md) (if needed)

@Pantani Pantani changed the title Feat/update sdk feat: update sdk Apr 17, 2023
Pantani added 3 commits April 17, 2023 20:25
# Conflicts:
#	app/app.go
#	testutil/keeper/initializer.go
#	testutil/keeper/keeper.go
#	testutil/keeper/monitoring_provider.go
#	x/project/module.go
#	x/project/module_simulation.go
#	x/project/simulation/simulation.go
@Pantani Pantani self-assigned this Apr 17, 2023
@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Merging #1011 (a74c25b) into main (991d025) will decrease coverage by 0.08%.
The diff coverage is 12.96%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1011      +/-   ##
==========================================
- Coverage   71.43%   71.35%   -0.08%     
==========================================
  Files         218      218              
  Lines        9475     9471       -4     
==========================================
- Hits         6768     6758      -10     
- Misses       2535     2540       +5     
- Partials      172      173       +1     
Impacted Files Coverage Δ
pkg/airdrop/airdrop_missions.go 0.00% <ø> (-50.00%) ⬇️
pkg/types/validator_set.go 92.30% <ø> (ø)
x/launch/keeper/msg_create_chain.go 93.18% <ø> (ø)
x/launch/keeper/msg_send_request.go 95.16% <ø> (ø)
x/launch/keeper/request.go 95.43% <ø> (ø)
x/launch/keeper/validator_set.go 100.00% <ø> (ø)
x/launch/simulation/simulation.go 0.00% <0.00%> (ø)
x/monitoringc/keeper/handshake.go 100.00% <ø> (ø)
x/monitoringc/keeper/monitoring_packet.go 100.00% <ø> (ø)
x/monitoringc/keeper/msg_create_client.go 95.23% <ø> (-0.15%) ⬇️
... and 32 more

... and 2 files with indirect coverage changes

@Pantani Pantani requested a review from ilgooz April 18, 2023 20:34
@Pantani Pantani marked this pull request as ready for review April 18, 2023 20:34
@Pantani Pantani changed the title feat: update sdk feat: bump cosmos-sdk to v0.47.1 Apr 18, 2023
testutil/gen_app.go Outdated Show resolved Hide resolved
testutil/gen_app.go Outdated Show resolved Hide resolved
jeronimoalbi
jeronimoalbi previously approved these changes Apr 26, 2023
Copy link
Collaborator

@jeronimoalbi jeronimoalbi left a comment

Choose a reason for hiding this comment

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

Great work @Pantani!

I left a couple of comments but otherwise LGTM 👍

One thing to comment is that cli-plugin-network must also be updated otherwise Ignite's "network" command won't work with the updated SPN.

Pantani and others added 2 commits April 26, 2023 14:26
Co-authored-by: Jerónimo Albi <jeronimo.albi@gmail.com>
@Pantani
Copy link
Collaborator Author

Pantani commented Apr 26, 2023

Great work @Pantani!

I left a couple of comments but otherwise LGTM 👍

One thing to comment is that cli-plugin-network must also be updated otherwise Ignite's "network" command won't work with the updated SPN.

the plugin will be the last one

@Pantani Pantani merged commit 36c7a9e into main Apr 26, 2023
@Pantani Pantani deleted the feat/update-sdk branch April 26, 2023 20:33
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.

2 participants