Skip to content

Fix opal-plus sync workflow: replace rebase with merge#891

Merged
EliMoshkovich merged 1 commit intomasterfrom
PER-14366-fix-opal-plus-sync-workflow
Apr 6, 2026
Merged

Fix opal-plus sync workflow: replace rebase with merge#891
EliMoshkovich merged 1 commit intomasterfrom
PER-14366-fix-opal-plus-sync-workflow

Conversation

@EliMoshkovich
Copy link
Copy Markdown
Contributor

Summary

  • Replaced the broken rebase-based sync with a merge-based approach
  • The old workflow rebased public-master onto opal/master then tried a regular push, which always fails because rebase rewrites history
  • New workflow merges opal/master into public-master (branched from opal-plus master), preserving opal-plus customizations (banner, Docker image names) and only bringing in new upstream changes incrementally
  • No force push needed — regular git push works since we only add merge commits

Test plan

  • Trigger workflow manually via workflow_dispatch and verify it completes successfully
  • Verify the PR created in opal-plus shows only new opal changes, not the full diff

🤖 Generated with Claude Code

The rebase-based sync was broken — rebase rewrites history, causing
regular push to be rejected (fetch first). Replaced with a merge-based
approach that preserves opal-plus customizations and only brings in
new upstream changes incrementally.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear bot commented Mar 31, 2026

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 31, 2026

Deploy Preview for opal-docs canceled.

Name Link
🔨 Latest commit 97d33b7
🔍 Latest deploy log https://app.netlify.com/projects/opal-docs/deploys/69cc2c0bc553a000082e485a

@EliMoshkovich EliMoshkovich merged commit 59b2180 into master Apr 6, 2026
12 checks passed
@EliMoshkovich EliMoshkovich deleted the PER-14366-fix-opal-plus-sync-workflow branch April 6, 2026 15:02
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