-
Notifications
You must be signed in to change notification settings - Fork 69
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
Refactor output format framework to support tables and empty format #1540
Refactor output format framework to support tables and empty format #1540
Conversation
updated instructions for generating changelog
retention bytes command -1 flag updated desc.
Co-authored-by: Wojciech Trocki <[email protected]>
updated Retention
WIP: Issues #1429 and 1440
endif | ||
app-services-go-linter -path ./pkg/core/localize/locales ./... | ||
.PHONY: lint-lang | ||
# lint-lang: ## Lint i18n files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this
endif | ||
app-services-go-linter -path ./pkg/core/localize/locales ./... | ||
.PHONY: lint-lang | ||
# lint-lang: ## Lint i18n files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this
@@ -1,30 +1,4 @@ | |||
|
|||
<a name="v0.39.0"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restore this changelog (generate it)
@@ -1,30 +1,4 @@ | |||
|
|||
<a name="v0.39.0"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restore this changelog (generate it)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing change. I did not verify this but code looks very good
We need to resolve conflicts, remove some extra changes. Then myself and @rkpattnaik780 can verify and merge this change |
Looks bit counter-intuitive to see "none" in
|
Motiviation
Addressing issue #1429
Changes made:
Verification Steps:
./rhoas kafka topic create --name=kannan --output=none