Skip to content

Conversation

jehelset
Copy link
Contributor

Closes #630

Copy link

copy-pr-bot bot commented Sep 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Collaborator

@ericniebler ericniebler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woohoo! thank you!

@ericniebler
Copy link
Collaborator

/ok to test 9dda21c

@jehelset
Copy link
Contributor Author

uh, i'll have a look...

@jehelset jehelset force-pushed the jeh/fix-test_concepts_receiver branch from 9dda21c to d282f87 Compare September 21, 2025 21:27
@jehelset
Copy link
Contributor Author

jehelset commented Sep 21, 2025

the noexceptness is no longer a part of the receiver-concept, rather statically asserted inside the completion-tag operator(), so i removed those tests, added some missing concept-aliases on the test-receivers, and made sure to use completion-signatures<...> rather than just int...

@ericniebler
Copy link
Collaborator

/ok to test d282f87

@ericniebler ericniebler merged commit 21ead1d into NVIDIA:main Sep 22, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix test_concepts_receiver.cpp
2 participants