Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new gh stack alias subcommand to install/remove a short wrapper executable (default gs) that forwards to gh stack, and wires it into the root command.
Changes:
- Register new
aliassubcommand in the root CLI. - Implement alias creation/removal via a marked wrapper script under
~/.local/bin. - Add tests covering alias name validation and wrapper install/remove behavior.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| cmd/root.go | Registers the new alias subcommand. |
| cmd/root_test.go | Updates subcommand registration expectations to include alias. |
| cmd/alias.go | Implements gh stack alias command, wrapper creation/removal, and helpers. |
| cmd/alias_test.go | Adds unit tests for alias validation and wrapper install/remove logic. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Warning This is an internal experiment to assess Copilot's ability to auto-approve PRs. Please 👍 this comment if the assessment below is correct and 👎 if not. Feedback in #f-ccr-auto-approve is appreciated! Copilot thinks this PR is not ready to approve — see review comments for details. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fewer keystrokes by aliasing
gstogh stack. Adds a helper command that will create (and remove) the alias for you.New alias command functionality:
aliassubcommand (AliasCmd) that lets users create or remove a shell alias (default:gs) forgh stack, installing a wrapper script in~/.local/bin/. The command validates alias names, checks for conflicts, and handles removal, with special instructions for Windows. (cmd/alias.go)aliascommand in the root command so it is available as a top-level CLI subcommand. (cmd/root.go)aliascommand, ensuring it is listed among available subcommands. (cmd/root_test.go)Testing and validation:
cmd/alias_test.go)