Skip to content

SP-1447: fix merge preview conflict reporting and branch docs - #402

Merged
Laberion Ajvazi (LaberionAjvazi) merged 2 commits into
mainfrom
LaberionAjvazi/sp-1447-fixes
Jul 31, 2026
Merged

SP-1447: fix merge preview conflict reporting and branch docs#402
Laberion Ajvazi (LaberionAjvazi) merged 2 commits into
mainfrom
LaberionAjvazi/sp-1447-fixes

Conversation

@LaberionAjvazi

Copy link
Copy Markdown
Contributor

Description

Follow-up fixes for the beta config branch commands, found while testing them end to end:

  • merge preview now counts nodes that conflict structurally, so a merge that merge apply would refuse is no longer summarised as clean.
  • --bump offers PATCH only, which is what the API supports.
  • Docs: corrected the CUSTOM resolution example, and documented that merge apply needs either -f or --sourceKey with --sourceVersion, and that create --sourceVersion does not accept LATEST.

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 91951de. Configure here.

Comment thread src/commands/configuration-management/branch/branch.command.service.ts Outdated
@sonarqubecloud

Copy link
Copy Markdown

@LaberionAjvazi
Laberion Ajvazi (LaberionAjvazi) marked this pull request as ready for review July 30, 2026 11:32
@LaberionAjvazi
Laberion Ajvazi (LaberionAjvazi) merged commit 63e15c5 into main Jul 31, 2026
6 checks passed
@LaberionAjvazi
Laberion Ajvazi (LaberionAjvazi) deleted the LaberionAjvazi/sp-1447-fixes branch July 31, 2026 13:30
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