Skip to content

Consider clang-format and clang-tidy support #30

Closed as not planned
Closed as not planned

Description

@alexrp

In a similar vein to:

Unfortunately, Zig does not currently ship with these tools, so we would have to require users to install them.

Questions:

  • Should we mandate a specific code style like zig fmt or let it be configurable?
  • Should we try to detect the presence of these tools on disk or just go ahead and invoke them if the user asked us to?
  • Should clang-tidy be tied to WarningLevel? Or should we maybe use AnalysisLevel/AnalysisMode?

Activity

  1. added
    state: deliberationIssues that require design work and/or discussion.
    area: sdkIssues related to the MSBuild SDK.
    area: samplesIssues related to sample projects.
    area: tasksIssues related to MSBuild tasks.
    on Jul 14, 2021
  2. added this to the v4.0 milestone on Jul 14, 2021
  3. self-assigned this
    on Jul 14, 2021
  4. alexrp commented on Jul 14, 2021

    @alexrp
    SponsorMemberAuthor
  5. alexrp commented on Jul 24, 2021

    @alexrp
    SponsorMemberAuthor
    • Code style: There's too much variation in the C/C++ world. Better to let it be configurable.
    • Tool detection: There's virtually no precedent for doing this in the MSBuild world, so I think users would find it surprising. We should just invoke the tools if asked to and error out appropriately if they're missing.

    Need to think more about where clang-tidy integration would fit in.

  6. modified the milestones: v4.0, v5.0 on Oct 30, 2022
  7. modified the milestones: v5.0, Future on Jan 1, 2024
  8. removed their assignment
    on Jan 27, 2024
  9. removed this from the Future milestone on Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: documentationIssues related to the documentation.area: samplesIssues related to sample projects.area: sdkIssues related to the MSBuild SDK.area: tasksIssues related to MSBuild tasks.state: deliberationIssues that require design work and/or discussion.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions