Skip to content

[AIR-3046] qs: eliminate unused funcs#137

Merged
host6 merged 1 commit into
untillpro:mainfrom
host6:AIR-3046-qs-eliminate-unused-funcs-pr
Feb 23, 2026
Merged

[AIR-3046] qs: eliminate unused funcs#137
host6 merged 1 commit into
untillpro:mainfrom
host6:AIR-3046-qs-eliminate-unused-funcs-pr

Conversation

@host6

@host6 host6 commented Feb 23, 2026

Copy link
Copy Markdown
Collaborator

@augmentcode

augmentcode Bot commented Feb 23, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR removes several unused helper functions from the gitcmds package (AIR-3046), reducing maintenance surface and trimming dependencies.

Changes:

  • Deleted merged-branch discovery and remote-branch deletion helpers (GetMergedBranchList, DeleteBranchesRemote) from gitcmds/gitcmds.go.
  • Removed the upstream fast-forward helper PullUpstream (and its related gh/git command pipeline usage) from gitcmds/gitcmds.go.
  • Dropped GitHub issue title lookup via gh issue view --json by removing GetGithubIssueNameByNumber from gitcmds/github.go.
  • Simplified OpenGitRepository error handling by always returning a wrapped open error.

Technical Notes: The removed code previously depended on external tools like gh and shell filters (gawk) for some workflows; remaining commands appear to rely on other existing git flows for repo sync/PR operations.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread gitcmds/gitcmds.go
@host6
host6 merged commit 6a9c344 into untillpro:main Feb 23, 2026
0 of 3 checks passed
@host6
host6 deleted the AIR-3046-qs-eliminate-unused-funcs-pr branch February 23, 2026 20:36
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.

1 participant