Skip to content

Conversation

@paulohtb6
Copy link
Contributor

Set the default language for rpk generate app to go since it's the only one available today.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v25.2.x
  • v25.1.x
  • v24.3.x

Release Notes

Improvements

  • On rpk generate app, set go as the default language.

Copilot AI review requested due to automatic review settings October 29, 2025 00:19
@paulohtb6 paulohtb6 requested review from a team, kbatuigas and r-vasquez as code owners October 29, 2025 00:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR sets a default language for the rpk generate app command to improve user experience. Previously, users had to explicitly specify a language, but now "go" is set as the default since it's currently the only supported language.

Update src/go/rpk/pkg/cli/generate/app.go

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
}

cmd.Flags().StringVarP(&langFlag, "language", "l", "", "The language you want the code sample to be generated with")
cmd.Flags().StringVarP(&langFlag, "language", "l", "go", "The language you want the code sample to be generated with. Available language: `go`.")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
cmd.Flags().StringVarP(&langFlag, "language", "l", "go", "The language you want the code sample to be generated with. Available language: `go`.")
cmd.Flags().StringVarP(&langFlag, "language", "l", "go", "The language you want the code sample to be generated with. Available language: 'go'")

We try to avoid using ``` in the terminal, and also, end the flag text without a period.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants