Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Allow easier CI runs for PRs from forks #330

Open
aguschin opened this issue Jul 1, 2022 · 0 comments
Open

Allow easier CI runs for PRs from forks #330

aguschin opened this issue Jul 1, 2022 · 0 comments
Labels
good first issue Good for newcomers 🏡 housekeeping Fighting technical debt and improving dev process p3-low Low priority testing MLEM testing

Comments

@aguschin
Copy link
Contributor

aguschin commented Jul 1, 2022

Currently, we need to manually approve each commit in PR from fork to be run in CI. This adds a lag into getting valuable feedback for external contributors.

Suggestion: split tests and CI into two parts: one that don't require secrets and the other that does. Then run the first each time, but still require manual approve to run the second part.
@aguschin aguschin added 🏡 housekeeping Fighting technical debt and improving dev process testing MLEM testing labels Jul 1, 2022
@aguschin aguschin added the good first issue Good for newcomers label Aug 31, 2022
@aguschin aguschin mentioned this issue Oct 16, 2022
Merged
@aguschin aguschin added the p3-low Low priority label Nov 8, 2022
@omesser omesser mentioned this issue Jul 13, 2023
Draft
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
good first issue Good for newcomers 🏡 housekeeping Fighting technical debt and improving dev process p3-low Low priority testing MLEM testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@aguschin