-
Notifications
You must be signed in to change notification settings - Fork 243
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
Export lists of changed files via action output parameter #20
Comments
Would this make it so I could get a list of files that changed that matched a specific filter? My use case is I'd like to pass a path separated list of files to
|
Unfortunately right now this wouldn't work I will change it so it could support your use case. Maybe I will get to it tomorrow. If not, then it will take some more time as I'm going for a vacation afterwards. Anyway, to avoid problems with files with white-space characters, the output will be definitely in the JSON format. |
Sounds good, thank you :) I'm not familiar with the |
Here's the proposal. I already implemented in a PR #32 - uses: dorny/paths-filter@export-files
id: filter
with:
list-files: shell
filters: |
markdown:
- added|modified: '*.md'
- name: Lint Markdown
if: ${{ steps.filter.outputs.markdown == 'true' }}
run: npx textlint ${{ steps.filter.outputs.markdown_files }} Input parameter Another part of the solution is the For linting it obviously doesn't make sense to consider removed files and linting tools would probably error on non-existent files. Would this work for you? |
This looks great! Yeah we would totally use this :) |
Implemented in release |
For some use-cases it might be useful to provide access to lists of changed (or added/deleted/modified) files via output parameters.
For this use-cases there is already
Changed Files Exporter
action. Benefit to implement it here is that we could make it work also for push events and provide different encoding option (JSON array, new-line delimited).The text was updated successfully, but these errors were encountered: