Skip to content

Revert "chore: try compare-two-pass-review on Claude PR reviews" - #13890

Merged
MounirDhahri merged 1 commit into
mainfrom
revert-13865-chore/try-two-pass-claude-review
Aug 6, 2026
Merged

Revert "chore: try compare-two-pass-review on Claude PR reviews"#13890
MounirDhahri merged 1 commit into
mainfrom
revert-13865-chore/try-two-pass-claude-review

Conversation

@MounirDhahri

Copy link
Copy Markdown
Member

Reverts #13865

@MounirDhahri MounirDhahri self-assigned this Aug 6, 2026
@MounirDhahri
MounirDhahri enabled auto-merge (squash) August 6, 2026 09:29
@MounirDhahri
MounirDhahri disabled auto-merge August 6, 2026 09:29
@MounirDhahri
MounirDhahri merged commit e6f6f0e into main Aug 6, 2026
10 of 15 checks passed
@MounirDhahri
MounirDhahri deleted the revert-13865-chore/try-two-pass-claude-review branch August 6, 2026 09:30
@claude

claude Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Reverts #13865, unpinning the AI PR review workflow from the claude/model-performance-comparison-n1wew6 trial branch back to artsy/duchamp/.github/workflows/claude-review.yml@main and dropping the compare-two-pass-review: true input. The revert is clean — the file now matches its pre-#13865 state exactly.

Issues Found

None. The only diff is the workflow ref and the with: block, both restored to the values #13865 changed. The missing trailing newline was there before this PR too, so it isn't introduced here.

Areas Reviewed

Nothing notable. No app code, no Relay artifacts, no UI. Reverting to @main means this repo picks up whatever duchamp ships on main, which is the intended default.

Questions for Author

Was the trial conclusive, or is this a rollback because artsy/duchamp#110 stalled? The #13865 comment said "keep compare-two-pass-review: true if the trial goes well" — worth a line in the description so the next person doesn't re-run the same experiment.

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.

1 participant