New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Only run CI checks when necessary (affected files changed) #835
Conversation
5755ca0
to
7424f6e
Compare
…ual workflow run - weird
.github/workflows/plan-examples.yml
Outdated
# if: "contains(github.event_name, 'pull_request')" | ||
# uses: actions/checkout@v3 | ||
# with: | ||
# ref: refs/pull/${{github.event.pull_request.number}}/merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Zvikan do you know why we need the two different routes for checking out? plans only run on PRs so the standard checkout should suffice, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to #572
"Explicit checkout PR code as pull_request_target changes checkout behavior to default to source main."
If we'll leave it with default checkout while still using pull_request_target
, it will use source main instead of the PR author branch & fork, so we need to explicitly checkout the PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok thanks - I put that back as it was
What does this PR do?
Motivation
More
pre-commit run -a
with this PRNote: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators
Additional Notes