Skip to content

[AIR-3027] qs: IamInMyBranch fucntionality is duplicated in few places#132

Merged
host6 merged 2 commits intountillpro:mainfrom
host6:AIR-3027-qs-IamInMyBranch-fucntionality-is-duplicated-in-fe-pr
Feb 23, 2026
Merged

[AIR-3027] qs: IamInMyBranch fucntionality is duplicated in few places#132
host6 merged 2 commits intountillpro:mainfrom
host6:AIR-3027-qs-IamInMyBranch-fucntionality-is-duplicated-in-fe-pr

Conversation

@host6
Copy link
Collaborator

@host6 host6 commented Feb 23, 2026

AIR-3027 qs: IamInMyBranch fucntionality is duplicated in few places
https://untill.atlassian.net/browse/AIR-3027

@augmentcode
Copy link

augmentcode bot commented Feb 23, 2026

🤖 Augment PR Summary

Summary: This PR deduplicates the “am I on main?” branch-detection logic across the QS CLI.

Changes:

  • Replaced the old IamInMainBranch helper with GetCurrentBranchInfo, returning current branch, main branch, and a precomputed isMain flag
  • Refactored callers (gitcmds.Download, commands.Dev, commands.U) to use the new helper instead of repeating GetCurrentBranchName + GetMainBranch + EqualFold
  • Added an archived uspec implementation note documenting the refactor plan and verification steps

Technical Notes: Centralizing the main-branch check reduces drift between commands and ensures consistent error wrapping for main-branch detection failures.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

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.

@host6 host6 merged commit b7c8752 into untillpro:main Feb 23, 2026
0 of 3 checks passed
@host6 host6 deleted the AIR-3027-qs-IamInMyBranch-fucntionality-is-duplicated-in-fe-pr branch February 23, 2026 09:58
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