Skip to content

feat: add ignoreLocalModule option - #68

Open
wreulicke wants to merge 3 commits into
tomarrell:masterfrom
wreulicke:feat/report-local-module
Open

wreulicke wants to merge 3 commits into
tomarrell:masterfrom
wreulicke:feat/report-local-module

Conversation

@wreulicke

@wreulicke wreulicke commented Sep 10, 2026 •

Copy link
Copy Markdown

Summary

  • Add the ignoreLocalModule configuration option.
  • Skip errors returned from other packages within the current Go module when set to true.
  • Keep the current package behavior controlled by reportInternalErrors.
  • Default ignoreLocalModule to false to preserve existing behavior.
  • Add documentation and analysis test cases.

Testing

  • go test ./... -count=1

Fixes #22

🤖 Generated with Claude Code

@wreulicke

Copy link
Copy Markdown
Author

I am not sure whether to name it ignore Local Module.

@tomarrell

Copy link
Copy Markdown
Owner

@wreulicke starting the CLI from NewDefaultConfig() stops a user's ignoreSigs from replacing the defaults, so with ignoreSigs: [".Errorf("] wrapcheck still ignores errors.New. Could you rename the option to ignoreLocalModule, default it to false, and revert the main.go change?

wreulicke and others added 2 commits October 15, 2026 10:30
Default to false and revert the main.go change so that a user's
ignoreSigs keeps replacing the defaults.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@wreulicke wreulicke changed the title feat: add reportLocalModule option feat: add ignoreLocalModule option Oct 15, 2026
@wreulicke

Copy link
Copy Markdown
Author

@tomarrell Thanks for the review!

Renamed the option to ignoreLocalModule (default false) and reverted the main.go change.

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.

Flag to ignore main module

2 participants