Skip to content

Testing: Check c(begin|end) members of C++20 ranges - #3612

Merged
Stephan T. Lavavej (StephanTLavavej) merged 13 commits into
microsoft:mainfrom
JMazurkiewicz:const-cxx20-views
Apr 7, 2023
Merged

Testing: Check c(begin|end) members of C++20 ranges#3612
Stephan T. Lavavej (StephanTLavavej) merged 13 commits into
microsoft:mainfrom
JMazurkiewicz:const-cxx20-views

Conversation

@JMazurkiewicz

@JMazurkiewicz Jakub Mazurkiewicz (JMazurkiewicz) commented Mar 31, 2023

Copy link
Copy Markdown
Contributor

Towards #3391.

Open
34 tasks
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added test Related to test code ranges C++20/23 ranges labels Mar 31, 2023
StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) requested changes Mar 31, 2023
View reviewed changes
Comment thread tests/std/tests/P0896R4_views_common/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_views_common/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_views_common/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_views_join/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_views_reverse/test.cpp Outdated
Comment thread tests/std/tests/P0896R4_views_reverse/test.cpp
Comment thread tests/std/tests/P0896R4_views_reverse/test.cpp Outdated
@JMazurkiewicz

This comment was marked as resolved.

Sign in to view
@StephanTLavavej

Stephan T. Lavavej (StephanTLavavej) commented Apr 7, 2023

Copy link
Copy Markdown
Member

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

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit adaf68c into microsoft:main Apr 7, 2023
@StephanTLavavej

Stephan T. Lavavej (StephanTLavavej) commented Apr 7, 2023

Copy link
Copy Markdown
Member

Thanks for extending this test coverage even more and ensuring that the STL remains high-quality! ✨ ✅ 😺

@JMazurkiewicz
Jakub Mazurkiewicz (JMazurkiewicz) deleted the const-cxx20-views branch April 7, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) StephanTLavavej approved these changes
+1 more reviewer
@strega-nil-ms nicole mazzuca (strega-nil-ms) strega-nil-ms approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej)

Labels

ranges C++20/23 ranges test Related to test code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JMazurkiewicz @StephanTLavavej @strega-nil-ms