Skip to content

Conversation

@DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Jan 26, 2023

That way, users will immediately get an error if a compiler cannot be found, instead of having to wait a long time before the error is thrown.

Replaces #769
Closes #769
Closes #677
Closes #727

@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Merging #770 (89caaef) into master (05f6854) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #770      +/-   ##
==========================================
+ Coverage   81.54%   81.59%   +0.05%     
==========================================
  Files           3        3              
  Lines         661      663       +2     
==========================================
+ Hits          539      541       +2     
  Misses        122      122              
Impacted Files Coverage Δ
src/PackageCompiler.jl 91.36% <100.00%> (+0.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@DilumAluthge
Copy link
Member Author

I'm not sure what the current state of CI is.

@IanButterworth Are these CI failures expected?

@IanButterworth
Copy link
Member

1.6 won't pass. The rest should but there are gremlins

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.

Check c++ compiler earlier Fail early if no compiler can be found?

2 participants