FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

CI: Add path filters to python-deps workflow by adityasharad · Pull Request #1152 · github/codeql-action · GitHub

CI: Add path filters to python-deps workflow - #1152

Merged
adityasharad merged 3 commits into
mainfrom
adityasharad/actions/python-deps-path-filters
Jul 28, 2022
Merged

adityasharad merged 3 commits into
mainfrom
adityasharad/actions/python-deps-path-filters

Conversation

adityasharad commented Jul 27, 2022
edited
Loading

Copy link
Copy Markdown
Contributor

No need to run this workflow on all PRs, only those that change the Python dependency installation mechanism.

Reviewer note: This will need an update to the branch protection rules, because these checks can no longer be required on all PRs.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • [N/A] Confirm the readme has been updated if necessary.
  • [N/A] Confirm the changelog has been updated if necessary.

adityasharad requested a review from a team as a code owner July 27, 2022 19:42
adityasharad marked this pull request as draft July 27, 2022 19:42
No need to run this workflow on all PRs, only those that
change the Python dependency installation mechanism.
adityasharad force-pushed the adityasharad/actions/python-deps-path-filters branch from 98a7225 to 0d16d70 Compare July 27, 2022 20:17

aeisenberg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe we should run this on a schedule as well (weekly?). And put in a workflow_dispatch trigger so we can run manually.

adityasharad marked this pull request as ready for review July 27, 2022 21:34
adityasharad enabled auto-merge July 28, 2022 21:48
adityasharad merged commit 6c23d76 into main Jul 28, 2022
adityasharad deleted the adityasharad/actions/python-deps-path-filters branch July 28, 2022 22:19
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL