Skip to content

fix: checkout PR head for app code, overlay action from main#14

Merged
DeDuckProject merged 1 commit intomainfrom
claude/optimize-pipeline-triggers-L1uZG
Mar 12, 2026
Merged

fix: checkout PR head for app code, overlay action from main#14
DeDuckProject merged 1 commit intomainfrom
claude/optimize-pipeline-triggers-L1uZG

Conversation

@DeDuckProject
Copy link
Copy Markdown
Owner

For issue_comment events we need both:

  • PR's code (to run the correct version of the app for recording)
  • Main's packages/action/ (which has the trigger handling logic)

Solution: checkout PR head ref first, then git checkout just packages/action/ from origin/main. The bundled dist/index.js includes all core trigger logic so no other files needed.

https://claude.ai/code/session_01F7qRXukhynYjUukzhaBzSK

For issue_comment events we need both:
- PR's code (to run the correct version of the app for recording)
- Main's packages/action/ (which has the trigger handling logic)

Solution: checkout PR head ref first, then git checkout just
packages/action/ from origin/main. The bundled dist/index.js
includes all core trigger logic so no other files needed.

https://claude.ai/code/session_01F7qRXukhynYjUukzhaBzSK
@github-actions
Copy link
Copy Markdown

GitGlimpse — Skipped

On-demand mode is enabled. Comment /glimpse on this PR to generate a demo.


Generated by git-glimpse

@DeDuckProject DeDuckProject merged commit a62be21 into main Mar 12, 2026
3 checks passed
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