-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
feat: run changed testsuites in PR ci #237
Conversation
…sable other steps to get faster feedback
…with empty change list
so this doesn't work if no test file has changed, and additionally it would also run if we just changed formatting etc. which would be wasteful. An alternative could be to use a similar system as in the vite repo by triggering runs on demand from Because we don't have to deal with an external repo, the application for auth would not be needed, however we'd have to add code from vite repo that limits these comments to triage role. |
as mentioned in the comment above the approach in this PR isn't working in all conditions. the comment trigger should be added in a separate PR |
lets see if this works 👀