-
Notifications
You must be signed in to change notification settings - Fork 248
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
Override option to set all Filters to true #204
Comments
It would be nice if we could append a string in the commit message to skip all check. e.g. This is handy when tweaking CI. |
Turns this was easy to implement, I had no idea you get access to the commit message from the github context: This example uses Something like this: skipChanges:
name: ⏭️ Skip changes check
runs-on: ubuntu-22.04
outputs:
isSkip: ${{ steps.skip.outputs.skip }}
steps:
- id: skip
run: |
echo "skip=$(echo ${{ contains(github.event.head_commit.message, '[skip checks]') }})" >> $GITHUB_OUTPUT
changes:
name: 🗂️ Check changed files
runs-on: ubuntu-22.04
needs: skipChanges
# Add it in the condition
outputs:
foo:
${{ needs.skipChanges.outputs.isSkip && 'true' || steps.filter.outputs.foo }}
steps:
- uses: actions/[email protected]
with:
fetch-depth: 20
- uses: dorny/paths-filter@v3
id: filter
with:
base: ${{ github.ref }}
filters: |
foo:
- 'foo/**/*.(ts|tsx)'
# Use `needs.changes.outputs` like you would normally
typecheck:
name: Typecheck foo
needs: changes
if: ${{ needs.changes.outputs.foo == 'true' }}
runs-on: ubuntu-22.04
steps:
- uses: actions/[email protected]
- uses: wyvox/action-setup-pnpm@v3
with:
args: '-F=foo'
- run: pnpm -F=foo run typecheck That checks the latest commit but you can check all commits too:
|
I am using a changes job to construct a list of lambda names which is passed to a lambda deployment job using a matrix strategy.
There are some cases where we would like to run all lambda deployments, regardless of whether or not they have changed. If there were an option to essentially mock as if all filters have detected changes, that would make this scenario very easy to deal with.
Is there a way to do this? Grateful for any advice.
The text was updated successfully, but these errors were encountered: