Conversation
Co-authored-by: Aiden Bai <aiden.bai05@gmail.com>
…co#678) Co-authored-by: Aiden Bai <aiden.bai05@gmail.com>
Co-authored-by: Aiden Bai <aiden.bai05@gmail.com>
Mondays (and on demand) it merges new commits from upstream millionco/react-doctor:main into our main, then opens/refreshes a main -> pinned PR for review. Uses GITHUB_TOKEN so the push to main does not trigger publish.yml/ci.yml; never auto-merges into pinned. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…illionco#658) Co-authored-by: Aiden Bai <aiden.bai05@gmail.com> Co-authored-by: Nisarg Patel <awesomenisarg@gmail.com>
gh pr commands in a fork default their base to the parent repo (millionco/react-doctor), so GITHUB_TOKEN-created PRs were denied with 'Resource not accessible by integration'. Pass --repo $GITHUB_REPOSITORY to pin every gh pr op to this fork. Also gate PR creation on main being ahead of pinned (rev-list) rather than only on a fresh upstream merge, so a failed PR creation is retried on the next run. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Gate PR on non-merge commits (--no-merges) so the gate matches the PR-body list and never opens a PR whose commit list renders empty. - Add 'set -euo pipefail' to the identity/fetch steps for consistency and to fail the job loudly if a fetch errors. - State emphatically in the PR body that the main->pinned PR must be merged with a merge commit (squash/rebase rewrites SHAs and makes the next sync reopen an ever-growing PR). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated upstream sync.
millionco/react-doctor@mainmainis synced up to upstream613058337673.Commits this PR brings onto
pinnedrn-animate-layout-propertyrule (Retirern-animate-layout-propertyrule millionco/react-doctor#677) (devin-ai-integration[bot])Merge with a merge commit — do not squash or rebase. Squashing rewrites these commits
under new SHAs, so the next sync would see them as unmerged and reopen an ever-growing PR.
A merge commit keeps
main's commits reachable frompinned, which is how thisworkflow detects that the sync is done.
Conflicts on the harden commit's files are expected and shown in the PR UI; resolve them in the merge.
Opened by the
sync-upstreamworkflow. Do not auto-merge.