Skip to content

ci: optimize jobs with path filtering #3074

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 21 commits into from
Jul 23, 2022
Merged

ci: optimize jobs with path filtering #3074

merged 21 commits into from
Jul 23, 2022

Conversation

ammario
Copy link
Member

@ammario ammario commented Jul 20, 2022

No description provided.

@kylecarbs
Copy link
Member

oof

@ammario ammario force-pushed the docs-test branch 2 times, most recently from 15540a6 to 8bf6720 Compare July 20, 2022 16:56
@ammario ammario requested a review from a team as a code owner July 20, 2022 17:15
@ammario ammario force-pushed the docs-test branch 2 times, most recently from 62795b7 to 93d159c Compare July 20, 2022 17:26
@jsjoeio
Copy link
Contributor

jsjoeio commented Jul 20, 2022

Less stars but was looking at this option too: https://github.com/marketplace/actions/changed-files

@jsjoeio
Copy link
Contributor

jsjoeio commented Jul 21, 2022

I was playing around with Changed Files and got it working in a small POC:

@ammario
Copy link
Member Author

ammario commented Jul 21, 2022

Yeah I'm struggling with my current approach for some reasons

@ammario
Copy link
Member Author

ammario commented Jul 21, 2022

I'll rip your method when I have a moment. Thanks for sharing @jsjoeio

@ammario ammario force-pushed the docs-test branch 4 times, most recently from 1c4cd78 to 2de8ff7 Compare July 23, 2022 19:12
@ammario ammario enabled auto-merge (squash) July 23, 2022 19:14
@ammario ammario requested review from kylecarbs and removed request for a team July 23, 2022 19:14
@ammario ammario disabled auto-merge July 23, 2022 20:45
@ammario ammario changed the title test change ci: fix docs skip Jul 23, 2022
@ammario ammario enabled auto-merge (squash) July 23, 2022 21:05
@ammario ammario disabled auto-merge July 23, 2022 21:09
@ammario ammario changed the title ci: fix docs skip ci: optimize jobs with path filtering Jul 23, 2022
@ammario ammario enabled auto-merge (squash) July 23, 2022 21:27
@ammario ammario merged commit d10513f into main Jul 23, 2022
@ammario ammario deleted the docs-test branch July 23, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kylecarbs kylecarbs kylecarbs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ammario @kylecarbs @jsjoeio