Skip to content

Conversation

@soniasingla
Copy link
Contributor

Signed-off-by: Sonia Singla soniasingla.1812@gmail.com

Rename --no-default-module to --no-module.

Fixes #1345

@soniasingla soniasingla changed the title rename(cmd): rename --no-default-module to --no-module improvement(cmd): rename --no-default-module to --no-module Jul 7, 2021
@ilgooz
Copy link
Member

ilgooz commented Jul 7, 2021

Hey @soniasingla, can you please update the integration/ tests as well to use this new name?

@soniasingla
Copy link
Contributor Author

soniasingla commented Jul 7, 2021

@ilgooz thanks i see they are still failing, can you please help me what i am doing wrong here?

Signed-off-by: Sonia Singla <soniasingla.1812@gmail.com>
@soniasingla
Copy link
Contributor Author

Re-running the tests fixed it i guess

Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

Thanks 👍
Also a quick note that once this PR is merged, we will also need to check if this option is used in the tutorials (which I believe this is the case for interchange tutorial) and we update this change on this PR cosmos/sdk-tutorials#708 that will be merge once Starport 0.17.0 is merge

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
@soniasingla soniasingla requested a review from lumtis July 7, 2021 14:38
@fadeev fadeev merged commit a6c8169 into ignite:develop Jul 9, 2021
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
)

* rename(cmd): rename --no-default-module to --no-module

Signed-off-by: Sonia Singla <soniasingla.1812@gmail.com>

* Update docs/cli/index.md

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>

Co-authored-by: Lucas Bertrand <lucas.bertrand.22@gmail.com>
Co-authored-by: Denis Fadeev <denis@fadeev.org>
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.

Rename --no-default-module to --no-module

5 participants