Skip to content

Remove inappropriate element from paths filters of "Check Packaging" workflow #44

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

Merged
merged 1 commit into from
Jan 11, 2022
Merged

Remove inappropriate element from paths filters of "Check Packaging" workflow #44

merged 1 commit into from
Jan 11, 2022

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Jan 11, 2022

The on.*.paths keys of GitHub workflows can be used to avoid unnecessary or unwanted workflow runs. This is done by
defining specific paths containing files relevant to the workflow's purpose. The workflow will only run when the trigger
event modifies a file matching the paths.

The "Check Packaging" workflow was copied from another project that uses the Task task runner tool In that project, it is appropriate for the workflow to be triggered on any change to the Task configuration file, and so it was added to the paths filter. This project uses npm instead of Task, and does not contain a taskfile, so the presence of that path in the filters is inappropriate and only adds unnecessary complexity to the workflow.

…workflow

The `on.*.paths` keys of GitHub workflows can be used to avoid unnecessary or unwanted workflow runs. This is done by
defining specific paths containing files relevant to the workflow's purpose. The workflow will only run when the trigger
event modifies a file matching the paths.

The "Check Packaging" workflow was copied from another project that uses the Task task runner tool. In that project, it
is appropriate for the workflow to be triggered on any change to the Task configuration file, and so it was added to the
`paths` filter. This project uses npm instead of Task, and does not contain a taskfile, so the presence of that path in
the filters is in appropriate and only adds unnecessary complexity to the workflow.
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: infrastructure Related to project infrastructure labels Jan 11, 2022
@per1234 per1234 self-assigned this Jan 11, 2022
@per1234 per1234 merged commit de654cc into arduino:main Jan 11, 2022
@per1234 per1234 deleted the clean-paths-filter branch January 11, 2022 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants