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

New ci #1792

Merged
merged 7 commits into from Jul 13, 2019
Merged

New ci #1792

merged 7 commits into from Jul 13, 2019

Conversation

fundies
Copy link
Contributor

@fundies fundies commented Jul 12, 2019

Update CI OS to ubuntu bionic
Reuse workers on CI
All Jobs run for debug and run mode now
Fixes emakes debug printing
fixes couple issues we had compiling things that were previously not covered by ci
fixes sed to remove color from logs on paste
print gcc version used in log

@RobertBColton RobertBColton added the Golden Ratio For exceptional commits demonstrative of superlative engineering quality. label Jul 12, 2019
@JoshDreamland JoshDreamland merged commit a131434 into enigma-dev:master Jul 13, 2019
fundies added a commit to fundies/enigma-dev that referenced this pull request Jul 17, 2019
Changes up the CI to reduce total load on Travis and put us in a better place to upgrade to C++14.
@RobertBColton
Copy link
Contributor

RobertBColton commented Jul 17, 2019

You removed my setting of early exit. Not sure if that matters since you moved all of it to scripts. The reason for it originally was because without it Travis continues trying to build even if some dependencies failed to be installed. Again, it's better to fail fast in that case, because without the dependencies there's no point wasting time trying to build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Golden Ratio For exceptional commits demonstrative of superlative engineering quality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants