Skip to content

Use non-strict matrices where possible#3015

Merged
StephanTLavavej merged 1 commit into
microsoft:mainfrom
CaseyCarter:strictly-ballroom
Aug 12, 2022
Merged

Use non-strict matrices where possible#3015
StephanTLavavej merged 1 commit into
microsoft:mainfrom
CaseyCarter:strictly-ballroom

Conversation

@CaseyCarter
Copy link
Copy Markdown
Contributor

@CaseyCarter CaseyCarter commented Aug 9, 2022

We've been a bit lax about using the strict_concepts_meow test matrices, which omit /permissive configs, when we don't really need to. Use the plain concepts_meow matrices instead for tests that pass with /permissive.

After this, the remaining strict tests are:

  • P0323R12_expected
  • P0896R4_common_iterator
  • P0896R4_views_drop
  • P0896R4_views_elements
  • P0896R4_views_filter
  • P0896R4_views_filter_death
  • P0896R4_views_filter_iterator
  • P0896R4_views_join
  • P0896R4_views_lazy_split
  • P0896R4_views_split
  • P0896R4_views_take
  • P0896R4_views_transform
  • P1522R1_difference_type
  • P1899R3_views_stride
  • P2441R2_views_join_with

We've been a bit lax about using the `strict_concepts_meow` test matrices which omit `/permissive` configs. Use the plain `concepts_meow` matrices instead for tests that pass with `/permissive`.
@CaseyCarter CaseyCarter added the test Related to test code label Aug 9, 2022
@CaseyCarter CaseyCarter requested a review from a team as a code owner August 9, 2022 20:40
@strega-nil-ms strega-nil-ms self-assigned this Aug 10, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

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

@StephanTLavavej StephanTLavavej merged commit 0a44c23 into microsoft:main Aug 12, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

No one can be told what the non-strict matrix is - you have to compile it for yourself. 🕶️ 🖥️ 💊

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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants