Skip to content

fix(commands): remove IsHidden property from UseGithubStorage option#1522

Open
joshjohanning wants to merge 2 commits intogithub:mainfrom
joshjohanning:joshjohanning/github-owned-storage-command-visibility
Open

fix(commands): remove IsHidden property from UseGithubStorage option#1522
joshjohanning wants to merge 2 commits intogithub:mainfrom
joshjohanning:joshjohanning/github-owned-storage-command-visibility

Conversation

@joshjohanning
Copy link

  • Did you write/update appropriate tests
  • Release notes updated (if appropriate)
  • Appropriate logging output
  • Issue linked
  • Docs updated (or issue created)
  • New package licenses are added to ThirdPartyNotices.txt (if applicable)

Description

Now that GitHub-owned storage is generally available, we shouldn't hide it when running --help.

image

Copilot AI review requested due to automatic review settings March 3, 2026 17:25
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

Exposes the --use-github-storage option in CLI help output now that GitHub-owned storage is generally available, aligning the gei and bbs2gh CLIs with current product availability.

Changes:

  • Unhides --use-github-storage in gei migrate-repo and gei generate-script.
  • Unhides --use-github-storage in bbs2gh migrate-repo and bbs2gh generate-script.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/gei/Commands/MigrateRepo/MigrateRepoCommand.cs Removes IsHidden so --use-github-storage appears in --help for gei migrate-repo.
src/gei/Commands/GenerateScript/GenerateScriptCommand.cs Removes IsHidden so --use-github-storage appears in --help for gei generate-script.
src/bbs2gh/Commands/MigrateRepo/MigrateRepoCommand.cs Removes hidden initializer so --use-github-storage appears in --help for bbs2gh migrate-repo.
src/bbs2gh/Commands/GenerateScript/GenerateScriptCommand.cs Removes IsHidden so --use-github-storage appears in --help for bbs2gh generate-script.

@@ -108,7 +108,6 @@ public MigrateRepoCommand() : base(
};
public Option<bool> UseGithubStorage { get; } = new("--use-github-storage")
{
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

This is a user-visible CLI change (the --use-github-storage flag will now appear in --help). Per the contribution guidelines, user-noticeable changes should include a bullet in RELEASENOTES.md; please add an entry describing that this option is no longer hidden / is now generally available.

Suggested change
{
{
IsHidden = true,

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Unit Test Results

    1 files      1 suites   10m 25s ⏱️
1 030 tests 1 030 ✅ 0 💤 0 ❌
1 031 runs  1 031 ✅ 0 💤 0 ❌

Results for commit 06be10c.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Code Coverage

Package Line Rate Branch Rate Complexity Health
ado2gh 71% 70% 737
bbs2gh 83% 78% 663
gei 81% 73% 608
Octoshift 84% 73% 1810
Summary 81% (7934 / 9819) 73% (1947 / 2649) 3818

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