8000 clang-tidy Action issues · Issue #2844 · MRtrix3/mrtrix3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
clang-tidy Action issues #2844
Open
Open
@Lestropie

Description

@Lestropie
  1. Is it possible to have the clang-tidy Action automatically hide any prior clang-tidy bot comments as "Outdated"? PRs can look quite spammy otherwise.

  2. The Action seems to run into hiccups whenever a force push is performed. It seems to take the warning & suggested change for a given line range for the out-of-date code, and propose that change to the same line range of the up-to-date code, which may not correspond (or in some cases the fix to the recommendation may have been implemented in the force push). I personally tend to force-push a bit because I don't like stacking commits to incrementally address issues that should have been addressed as a cohesive changeset. There's arguments for and against. But it's annoying regardless.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0