Skip to content

Bug 643397: Miapp uses a stale RepoBranchName - #9701

Closed
AleksanderGladkov wants to merge 1 commit into
mainfrom
bugs/643397-master-Miapp-RepoBranchName
Closed

Bug 643397: Miapp uses a stale RepoBranchName#9701
AleksanderGladkov wants to merge 1 commit into
mainfrom
bugs/643397-master-Miapp-RepoBranchName

Conversation

@AleksanderGladkov

@AleksanderGladkov AleksanderGladkov commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

ISSUE:
Initialize NAV enlistment, cd App/BCApps, run Invoke-Miapp.
It fails with the error: Exception: fatal: HEAD does not point to a branch
Miapp fails in BCApps when RepoBranchName contains a stale value from another repository, such as master from NAV.

CAUSE:
Miapp trusts the environment value without checking whether the branch exists on the current repository's origin.

SOLUTION:
Preserve valid explicit base branches. When the configured branch does not exist on origin, show a warning and fall back to the current repository's origin/HEAD.

TESTS:
Validated valid, unset, and stale base-branch values. Verified that Invoke-Miapp automatically uses main when the inherited value is master.

Fixes AB#643397

@AleksanderGladkov
AleksanderGladkov requested review from a team July 23, 2026 16:02
@github-actions github-actions Bot added Build: scripts & configs Build scripts and configuration files Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction labels Jul 23, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 23, 2026
@AleksanderGladkov

Copy link
Copy Markdown
Contributor Author

Partially solved in
#9713

The solution in the PR 9713 will work in master, but will not work in releases/28.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build: scripts & configs Build scripts and configuration files Other GitHub request for other area than SCM, Finance or Integration Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants