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

Add clang-tidy as a Visual Studio code analyser option #2186

Closed
wants to merge 1 commit into from
Closed

Add clang-tidy as a Visual Studio code analyser option #2186

wants to merge 1 commit into from

Conversation

theComputeKid
Copy link
Contributor

What does this PR do?

This PR enables the selection of clang-tidy as a code analyser when the code analyser action of Visual Studio is run in the IDE. Further details can be found in the official documentation.

How does this PR change Premake's behavior?

Add configuration-scoped option clangtidy for Visual Studio 2019 and above.

I have tried to follow pre-existing conventions:

  • The option name was a lowercase boolean based on compilebuildoutputs as an example.
  • The test_output_props tests for vs2019 were based on vs2022 test_output_props.
  • A test was also added to ensure pre-vs2019 configs were not affected.
  • Documentation was added based on the buildstlmodules example, which was a similar boolean.

Did you check all the boxes?

  • Focus on a single fix or feature; remove any unrelated formatting or code changes

  • Add unit tests showing fix or feature works; all tests pass

  • Mention any related issues (put closes #XXXX in comment to auto-close issue when PR is merged)

  • Follow our coding conventions

  • Minimize the number of commits

  • Align documentation to your changes

CUDA support has been queried multiple times (#2168, #1898 etc.). I have been maintaining this module for quite some time, so listing it here would let people use CUDA with Visual Studio (and on Linux).

Signed-off-by: thecomputekid <[email protected]>
@theComputeKid theComputeKid deleted the patch-1 branch February 17, 2024 13:36
@theComputeKid
Copy link
Contributor Author

Superceeded by #2187.

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.

1 participant