Skip to content
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

Move Changes action locally #7223

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

edmundmiller
Copy link
Contributor

@edmundmiller edmundmiller commented Dec 13, 2024

Attempting to move detect changes action to a local action in this repo.

#7007 (comment)

All good - complexity is bordering on too much now. It's a shame we can't redirect individual tests to specific hardware, you know...like a certain piece of software does very well.

This manages most of the issues well. @adamrtalbot

On the complexity, after we merge this one can we migrate the action to live in this repo? I think that structure works better for the mono-repo so these PRs aren't so complicated to test/hack on. @edmundmiller

@mashehu
Copy link
Contributor

mashehu commented Dec 13, 2024

What is the reason for making detect-changes local?

@edmundmiller
Copy link
Contributor Author

What is the reason for making detect-changes local?

I added some more context; sorry! I just wanted to get the ball rolling because @maxulysse asked about it in Slack.

@edmundmiller edmundmiller self-assigned this Dec 13, 2024
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