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

Remove jobs dependency in CI #2108

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Remove jobs dependency in CI #2108

merged 1 commit into from
Apr 25, 2024

Conversation

tokatoka
Copy link
Member

because it's slow.

@addisoncrump
Copy link
Collaborator

It's slow, but it also prevents us from spinning up all the fuzzers when the other CI checks don't pass. It makes it more parallelisable, at the cost of some individual PR times. I expect this will make most CI checks much slower as a result...

@tokatoka
Copy link
Member Author

tokatoka commented Apr 25, 2024

no this just removes the two dependency from ubuntu-doc and ubuntu-doc-test. we have 100 jobs. wasting 2 doesn't matter anyway?

@addisoncrump
Copy link
Collaborator

Ahhh I thought this was for fuzzers not the other two. I cannot read

@tokatoka tokatoka merged commit abcb2bf into main Apr 25, 2024
100 checks passed
@tokatoka tokatoka deleted the dont_dep branch April 25, 2024 14:07
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.

None yet

2 participants