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

[FEA]: Relax Catch2 header order requirements #600

Closed
1 task done
gevtushenko opened this issue Oct 23, 2023 · 0 comments · Fixed by #601
Closed
1 task done

[FEA]: Relax Catch2 header order requirements #600

gevtushenko opened this issue Oct 23, 2023 · 0 comments · Fixed by #601
Assignees
Labels
feature request New feature or request.

Comments

@gevtushenko
Copy link
Collaborator

Is this a duplicate?

Area

CUB

Is your feature request related to a problem? Please describe.

Currently, we require that all CUB headers in Catch2 tests are placed before the helper header. This requirement comes from a cudafe bug. We have to revisit this issue to see if the warning suppression is still required.

Describe the solution you'd like

No header order is required.

Describe alternatives you've considered

No response

Additional context

No response

@gevtushenko gevtushenko added the feature request New feature or request. label Oct 23, 2023
@github-project-automation github-project-automation bot moved this to Todo in CCCL Oct 23, 2023
@gevtushenko gevtushenko self-assigned this Oct 23, 2023
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Oct 23, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant