Fix the CI tests #625
Open
Fix the CI tests #625
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The CI failed because it could not find ut version 1.1.
Emscripten cannot compile the parameterized test because it does not recognise std::views::iota.
Pip do not want to install packages without it.
It is probably a bug in Apple Clang, the program crash when std::source_location is used in the tests.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem:
Solution:
--break-system-packageshad to be added to the command or it did not work. It could be better to use brew instead but I do not know if the packages exist and I do not think it is a big issue to use --break-system-packages in a CI build.std::source_locationseems to be broken for Apple Clang so its use is prevented if the comiler is Apple Clang.Issue: #
Reviewers:
@