Skip to content

chore: real merge commit to unblock PR #57 - #59

Merged
saadqbal merged 37 commits into
developfrom
resolve/pr-57-merge
Apr 24, 2026
Merged

chore: real merge commit to unblock PR #57#59
saadqbal merged 37 commits into
developfrom
resolve/pr-57-merge

Conversation

@saadqbal

@saadqbalsaadqbal commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Creates a real merge commit of main into develop so that main becomes a true ancestor. This unblocks PR #57.

Why PR #58 wasn't enough

PR #58 did merge main into develop, but it was squash-merged — squash collapses both parents into a single linear commit, so main's commits never joined develop's ancestry. Git therefore keeps finding a common ancestor of develop and main that has neither branch's current README content, and PR #57 keeps showing the same conflict.

Conflict resolution

Single file: README.md. Kept develop's version — it's main's post-rewrite short README plus the ## Security section added in #44. No content lost.

⚠️ MERGE INSTRUCTIONS

Use "Create a merge commit" when merging this PR. Do NOT squash, do NOT rebase. Either will strip the merge-commit parent and reproduce the exact problem this PR exists to fix.

If your repo default is squash-merge, override it for this PR via the dropdown on the green merge button.

Test plan

  • git merge --no-ff origin/main off develop conflicts only on README.md.
  • Resolved README keeps the ## Security section.
  • After merge (with --no-ff), git log --graph origin/develop..origin/main should be empty — proving main is now an ancestor.

Note

Low Risk
No functional code changes; this is intended to fix branch ancestry/merge history, so risk is limited to potential workflow/merge-management impacts.

Overview
This PR introduces no content/code changes in the diff; it is intended to land as a true merge commit of main into develop so Git history reflects main as an ancestor and downstream PRs stop repeatedly surfacing the same conflict (notably around README.md).

Reviewed by Cursor Bugbot for commit 8b4823d. Bugbot is set up for automated code reviews on this repo. Configure here.

optional runtime class name for GPU joBs in k3s
Update Helm chart versions to 1.0.0 for aks, bm, and eks; increment a…ppVersion to 1.24.10, update creation timestamps and digests in index.yaml.
Update Helm chart versions to 1.0.1 for aks, bm, and eks;
Update Helm chart versions to 1.0.2 for aks, bm, and eks
Update Helm chart versions to 1.0.2 for aks, bm, and eks
Update Helm chart versions to 1.0.3 for aks, bm, and eks; increment appVersion to 1.24.13
Update Helm chart version to 2.0.1 for tracebloc,
Update index.yaml with new creation timestamps for existing Helm charts
Refactor Helm chart release workflow to improve packaging and commit logic
Update Helm chart version from 2.0.1 to 1.0.0, reflecting a major version change for tracebloc
Enhance Helm chart release workflow by adding cleanup step for local chart packages.
Update Helm chart version to 1.0.1 and appVersion to 1.24.15
Update Helm chart release workflow to use 'client' naming convention
Rename Helm chart from 'tracebloc' to 'client' in Chart.yaml for consistency with new naming convention.
Enhance MySQL deployment configuration in Helm chart
Update Helm chart version to 1.0.1 and appVersion to 1.0.1 for client
Add installation of Tracebloc client Helm chart to script (#30)
improved logging and messaging
Refactor Docker status check in install-k8s.ps1
saadqbaland others added 7 commits March 6, 2026 23:06
chore: add narrow CODEOWNERS for security-sensitive paths
PR #58 merged main -> develop earlier, but was squash-merged. Squash
drops the merge-commit parent link, so main's commits never joined
develop's ancestry. That means PR #57 (develop -> main) keeps re-
surfacing the same README conflict: git finds a common ancestor that
has neither branch's current README content.
This PR creates a real merge commit so that main becomes a true
ancestor of develop. Once this lands, PR #57 should have no conflicts.
Conflict resolved: README.md — kept develop's version (which is main's
post-rewrite README + the `## Security` section added in #44).
IMPORTANT FOR MERGER: merge this PR using "Create a merge commit".
Do NOT squash and do NOT rebase — either of those will strip the
merge-commit parent and reproduce the exact problem this PR exists
to fix.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for freeto 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.

3 participants

@saadqbal@aptracebloc@LukasWodka