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

fix: add missing verbose mode flags #4286

Merged
merged 4 commits into from
Aug 2, 2024
Merged

fix: add missing verbose mode flags #4286

merged 4 commits into from
Aug 2, 2024

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Aug 2, 2024

We are using the verbose flag in some parts of the code, but the flag is no set

@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs component:cmd labels Aug 2, 2024
@Pantani Pantani added the backport/v28.x.y Backport to v28.x.y label Aug 2, 2024
Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

utACK

@Pantani Pantani merged commit 3152cf5 into main Aug 2, 2024
44 checks passed
@Pantani Pantani deleted the fix/chain-verbose-mode branch August 2, 2024 12:22
mergify bot pushed a commit that referenced this pull request Aug 2, 2024
* add missing verbose mode flags

* add changelog

* remove unused debug flag

(cherry picked from commit 3152cf5)

# Conflicts:
#	ignite/cmd/chain_lint.go
julienrbrt pushed a commit that referenced this pull request Aug 5, 2024
* fix: add missing verbose mode flags (#4286)

* add missing verbose mode flags

* add changelog

* remove unused debug flag

(cherry picked from commit 3152cf5)

# Conflicts:
#	ignite/cmd/chain_lint.go

* Update changelog.md

* remove unused file

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:cmd component:configs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants