docs: record the pull request commit instead of the merge commit [ENG-934] - #84
Conversation
…f the merge commit @currents/playwright 2.5.1 and @currents/cmd 1.11.0 (@currents/commit-info 1.1.0) record the last commit of the pull request when CI checks out a merge commit, so the ref: head.sha checkout is no longer needed to get the commit message. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LdJrRhBLEH9Uwt97JMGVTR
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Warning Review limit reached
This review includes 2 billable files and costs up to $0.50.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Or wait 41 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Your 62 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (2)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (1)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughThe documentation now describes how supported reporter versions handle pull-request merge commits, when they fetch the pull-request commit, and how GitHub Actions checkout settings affect the recorded commit. ChangesPull-request commit documentation
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to The documentation can be published without waiting for additional package releases. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
| Topic | Details | |||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| PR commit capture | Explain how reporters select the pull request commit across CI providers, validate it as a merge parent, fetch it when missing, and fall back safely when fetching is disabled or fails.Modified files (2)
Latest Contributors(2)
| |||||||||
| Workflow guidance | Update GitHub Actions examples to rely on current reporter behavior while documenting the legacy checkout required by older reporters and cypress-cloud.Modified files (2)
Latest Contributors(2)
|
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LdJrRhBLEH9Uwt97JMGVTR
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LdJrRhBLEH9Uwt97JMGVTR
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LdJrRhBLEH9Uwt97JMGVTR
Documents that the reporters record the last commit of the pull request when CI checks out a merge commit, and removes the
ref: ${{ github.event.pull_request.head.sha }}checkout that we recommended to get the right commit message.persist-credentials: falsefailure case, andCURRENTS_DISABLE_HEAD_COMMIT_FETCH. Therefcheckout stays forcypress-cloudand earlier reporter versions.ref: head.shafrom the example workflow.Merge after
@currents/playwright2.5.1and@currents/cmd1.11.0are on npmlatest. Both versions are what release-it will pick for the pendingfix:changes (currents-dev/currents-playwright#971, currents-dev/currents-reporter#421). Check them at release.Code: currents-dev/commit-info#9. Linear: ENG-934.
🤖 Generated with Claude Code
https://claude.ai/code/session_01LdJrRhBLEH9Uwt97JMGVTR
Summary by CodeRabbit
@currents/playwright2.5.1 and@currents/cmd1.11.0, pull request runs can record the source commit instead of the CI-created merge commit when its SHA is available. If needed, the reporter attempts to fetch the source commit; if fetching fails, it records the merge commit. You can disable fetching withCURRENTS_DISABLE_HEAD_COMMIT_FETCH=true.