feat: add ParamChange to ignite n request list/show
#3233
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: aljo242 <[email protected]>
aljo242
requested review from
lumtis,
fadeev,
jeronimoalbi,
tbruyelle and
Pantani
as code owners
|
Visit the preview URL for this PR (updated for commit 9e90b52): https://igntservices-docs--pr3233-feat-request-list-pa-dyjs9r4b.web.app (expires Mon, 12 Dec 2022 20:37:23 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 95379efd94dd497aaa37c2d0354e6e2cafca5ec5 |
Codecov ReportBase: 19.98% // Head: 19.98% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## chore/update-spn #3233 +/- ##
==================================================
Coverage 19.98% 19.98%
==================================================
Files 382 383 +1
Lines 30108 30220 +112
==================================================
+ Hits 6016 6039 +23
- Misses 23518 23601 +83
- Partials 574 580 +6
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
I think you reference the wrong number, #3219 it is from my PR, not the issue |
Fixed! thank you |
tbruyelle
approved these changes
Dec 6, 2022
aljo242
added a commit
that referenced
this pull request
Dec 8, 2022
* update spn * import * fix * update mocks * rename test * feat(`network`): `param-change` (#3142) * utd * add command * typo * add test * changelog * format * Update ignite/cmd/network_request_param_change.go Co-authored-by: Lucas Bertrand <[email protected]> * Update changelog.md Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: Lucas Bertrand <[email protected]> * refactor: rename command `param-change` into `change-param` (#3169) * change name * rename file * chore: upgrade `spn` (#3177) * gomod * gosum * endpoints (#3196) * format * request list * feat: add `ParamChange` to `ignite n request list/show` (#3233) * chore: go formatting (#3232) Co-authored-by: aljo242 <[email protected]> * request list * format * ascii Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: aljo242 <[email protected]> * format * feat(`network`): param-change apply to genesis (#3208) * stub out * prepare logic * fix test * Update ignite/services/network/networkchain/prepare.go Co-authored-by: Lucas Bertrand <[email protected]> * refactor WithKeyValueByte * add detail to error message * const format string * format * add integration test and fix * add check to command * format * finish check code * improve error msg * wip * fix * format * remove incorrect test * Update ignite/pkg/cosmosutil/genesis/genesis.go Co-authored-by: Danilo Pantani <[email protected]> * rename according to review * optimize appylParamChanges * basic ModuleParamField test * format * revert errors Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: Danilo Pantani <[email protected]> * add test * add test * flesh out genesis information tests * format Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: aljo242 <[email protected]> Co-authored-by: Danilo Pantani <[email protected]>
Jchicode
pushed a commit
to Jchicode/cli
that referenced
this pull request
Aug 9, 2023
* update spn * import * fix * update mocks * rename test * feat(`network`): `param-change` (ignite#3142) * utd * add command * typo * add test * changelog * format * Update ignite/cmd/network_request_param_change.go Co-authored-by: Lucas Bertrand <[email protected]> * Update changelog.md Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: Lucas Bertrand <[email protected]> * refactor: rename command `param-change` into `change-param` (ignite#3169) * change name * rename file * chore: upgrade `spn` (ignite#3177) * gomod * gosum * endpoints (ignite#3196) * format * request list * feat: add `ParamChange` to `ignite n request list/show` (ignite#3233) * chore: go formatting (ignite#3232) Co-authored-by: aljo242 <[email protected]> * request list * format * ascii Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: aljo242 <[email protected]> * format * feat(`network`): param-change apply to genesis (ignite#3208) * stub out * prepare logic * fix test * Update ignite/services/network/networkchain/prepare.go Co-authored-by: Lucas Bertrand <[email protected]> * refactor WithKeyValueByte * add detail to error message * const format string * format * add integration test and fix * add check to command * format * finish check code * improve error msg * wip * fix * format * remove incorrect test * Update ignite/pkg/cosmosutil/genesis/genesis.go Co-authored-by: Danilo Pantani <[email protected]> * rename according to review * optimize appylParamChanges * basic ModuleParamField test * format * revert errors Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: Danilo Pantani <[email protected]> * add test * add test * flesh out genesis information tests * format Co-authored-by: Lucas Bertrand <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: aljo242 <[email protected]> Co-authored-by: Danilo Pantani <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #3216