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

Enable __cpp_lib_concepts for EDG, part 2 #4297

Merged
merged 4 commits into from
Feb 6, 2024

Conversation

cpplearner
Copy link
Contributor

After #4296, concepts_20_matrix.lst and concepts_latest_matrix.lst will be near-identical to their usual counterparts. This PR changes the tests to include the usual matrices instead of the concepts ones, and then removes the concepts matrices.

Blocked on #4296.

@CaseyCarter CaseyCarter added test Related to test code blocked Something is preventing work on this labels Jan 4, 2024
@StephanTLavavej StephanTLavavej removed the blocked Something is preventing work on this label Feb 1, 2024
@cpplearner cpplearner marked this pull request as ready for review February 2, 2024 01:05
@cpplearner cpplearner requested a review from a team as a code owner February 2, 2024 01:05
@StephanTLavavej StephanTLavavej self-assigned this Feb 2, 2024
@StephanTLavavej StephanTLavavej removed their assignment Feb 2, 2024
@StephanTLavavej StephanTLavavej self-assigned this Feb 5, 2024
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit d008413 into microsoft:main Feb 6, 2024
35 checks passed
@StephanTLavavej
Copy link
Member

Thanks again for doing this matrix math! 🧮 😹 🎉

@cpplearner cpplearner deleted the edg-concepts-2 branch February 6, 2024 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Related to test code
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants