Skip to content

Conversation

@kolyshkin
Copy link
Collaborator

This is a followup to #186.

This repo:

  • does not have any paths that are excluded;
  • does not have any generated code.

Also, neither "common-false-positives" nor "legacy" presets suppress
any warnings, so remove those as well.

This repo:
 - does not have any paths that are excluded;
 - does not have any generated code.

 Also, neither "common-false-positives" nor "legacy" presets suppress
 any warnings, so remove those as well.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@thaJeztah
Copy link
Member

Thanks! I wanted to dig into these! I assumed the migration would keep the same config as we used before (just in "v2" format), but I wasn't exactly sure on all of them. Slightly lazy on my side, because I wanted to have CI "refreshed" before tagging atomicwriter, but did want to dig deeper. Thanks!!

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! 🙏

@thaJeztah thaJeztah merged commit c9593b2 into moby:main Apr 8, 2025
20 checks passed
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.

2 participants