Fix C&P issue int test of bool-likes in optional#149
Merged
Conversation
This was likely an omission by copying `MyBool`
Temporary workaround to make CI pass
Member
|
This is to acknowledge that I see the request. I will be able to look at it within 8 hours. |
Member
Is enough that I swap contents of this .github/workflows/ci.yml file? |
Contributor
Author
Yes. If you need customization there is a large number of inputs available. You might want to look int the GCC 4.8 failure which is new in this test Error |
This file contains hidden or 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
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.
This was likely an omission by copying
MyBoolin 1fb60cbCurrently it is exactly the same so provides not further coverage
If the bool-operator should also be explicit that needs to be added too
@akrzemi1
BTW: I'd suggest switching to the reusable Boost.CI workflow There the Clang-16 failure is fixed by using the container