-
Notifications
You must be signed in to change notification settings - Fork 12
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
ci: split core #1419
ci: split core #1419
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR leaves android and ubuntu in the same bed, I can split them up to if you like this work.
I like this work :)
0626617
to
7b81052
Compare
Seeing CI being tidied up always brings me joy :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I share the feeling. We deserved that.
Reviewable status: 0 of 7 files reviewed, 1 unresolved discussion (waiting on @octol and @zaneschepke)
.github/workflows/ci-nym-vpn-core-linux.yml
line 36 at r1 (raw file):
Previously, octol (Jon Häggblad) wrote…
I strongly suspect that we don't need triplet anymore, on any of the ci core workflows
About to remove it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 8 files reviewed, 1 unresolved discussion (waiting on @octol and @zaneschepke)
.github/workflows/ci-nym-vpn-core-linux.yml
line 36 at r1 (raw file):
Previously, pronebird (Andrej Mihajlov) wrote…
About to remove it.
Done.
@zaneschepke please have a look, provide your feedback. |
Extract Mac, iOS, Linux, Android workflows from core CI workflow into individual ones.
New workflows
Removed
This change is