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.

Sign up for GitHub

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

Jump to bottom

ci: build all #51

Merged
merged 2 commits into from
Jan 26, 2022
Merged

ci: build all #51

merged 2 commits into from
Jan 26, 2022

Conversation

adlerjohn
Copy link
Contributor

No description provided.

@adlerjohn adlerjohn self-assigned this Jan 22, 2022
@iqdecay
Copy link
Contributor

iqdecay commented Jan 22, 2022 •
edited
Loading

this will reduce compilation time? (building everything from the get-go => don't build them for every test?)

@adlerjohn
Copy link
Contributor Author

this will reduce compilation time?

I don't think so? Presumably it was being built on the first test only. I guess we'll find out.

@adlerjohn adlerjohn marked this pull request as draft January 22, 2022 20:18
@adlerjohn adlerjohn mentioned this pull request Jan 22, 2022
Closed
@adlerjohn adlerjohn marked this pull request as ready for review January 26, 2022 00:37
@adlerjohn
Copy link
Contributor Author

cc @digorithm

@adlerjohn adlerjohn merged commit 8eb5795 into master Jan 26, 2022
@adlerjohn adlerjohn deleted the adlerjohn/ci_build_all branch January 26, 2022 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@digorithm digorithm digorithm approved these changes

@iqdecay iqdecay Awaiting requested review from iqdecay

Assignees

@adlerjohn adlerjohn

Labels
None yet
Projects
Fuel Network
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adlerjohn @iqdecay @digorithm