Skip to content

🔧 Simplify secrets set to use single --repo flag#17977

Merged
dsyme merged 2 commits intomainfrom
or
Feb 23, 2026
Merged

🔧 Simplify secrets set to use single --repo flag#17977
dsyme merged 2 commits intomainfrom
or

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Feb 23, 2026

Summary

  • Replace separate --owner and --repo flags with a single --repo owner/repo flag for gh aw secrets set
  • Default to the current repository when --repo is not specified
  • Update documentation and usage examples to reflect the new flag interface

Copilot AI review requested due to automatic review settings February 23, 2026 20:17
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 pull request simplifies the gh aw secrets set command by replacing separate --owner and --repo flags with a single --repo owner/repo flag, improving usability and consistency with other commands in the codebase.

Changes:

  • Replaced --owner and --repo flags with a single --repo flag that accepts owner/repo format
  • Updated command to default to the current repository when --repo is not specified
  • Refactored repository determination logic to use the standardized addRepoFlag() helper

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pkg/cli/secret_set_command.go Replaced dual flag variables with addRepoFlag() call, simplified repository determination logic to handle single --repo flag with owner/repo format
docs/src/content/docs/setup/cli.md Updated documentation examples and options to reflect the new single --repo flag interface

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dsyme dsyme merged commit bf87e01 into main Feb 23, 2026
93 checks passed
@dsyme dsyme deleted the or branch February 23, 2026 20:22
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.

2 participants