Enable code analysis on all projects and resolve warnings · Microsoft/calculator#693

(2 评论) (0 反应) (0 负责人)C++ (16,196 star) (2,422 fork)batch import

Pri: 2approvedbuildcodebase qualityhelp wanted

描述

Visual Studio's code analysis tools can help find issues in our C++ code. Eventually, it would be great to have these tools running in all of our builds and to fix all the warnings they generate.

See #689 for an example of how to enable the tools during the build. We'll enable code analysis only in the "Release" configuration to avoid slowing down Debug builds.

To see which warnings still need to be fixed, check a recent build from the master branch.

It would be best to submit fixes for unrelated warnings as separate PRs. I'll close out this issue when 1) all C++ projects are building with code analysis rules on and 2) all the warnings generated by those tools are fixed or silenced.

贡献者指南

技术栈: cpp
领域: toolingbuild systemci cd
议题类型: feature
难度: 3
预计时间: over 1 week
活动状态: stale
清晰度: clear
前置要求: C++ basicsVisual Studio build configuration
新手友好度: 40
研究方向: Review issue #689 for an example of enabling code analysis during build. Examine the Release configuration in project files. Check the Azure DevOps build link to see current warnings. Plan to fix warnings in separate PRs as suggested.