Tom Arrell

Results 29 comments of Tom Arrell

G'day, I think both of these cases could be implemented neatly within this linter as it already has the definition checking there. As far as timeline, I can't give anything...

That should be possible, will just need a bit of a tweak to the AST matcher. I'll take a look when I get some time. If you want it sooner,...

Thanks for the report. Will have a look when I get some time.

G'day @Southclaws, cheers for the report. I've done some digging, and have a suggestion. Would you be able to try the config below? ```yaml ignorePackageGlobs: - "*github.com/Southclaws/storyden/pkg*" ``` I suspect...

@Southclaws, would you be able to try out the above?

Thanks for the info @hackerwins, I'll look at reproducing your case and pushing a fix soon.

@hackerwins @Southclaws I've just pushed release [v2.7.0](https://github.com/tomarrell/wrapcheck/releases/tag/v2.7.0) which makes this configuration value apply to functions called through interfaces, hopefully making it a bit more intuitive. Let me know if you...

@hackerwins no worries, it should be out in the next release. The update PR has already been merged into golangci-lint https://github.com/golangci/golangci-lint/pull/3287

I've brought this up again over at Pandoc. It's not entirely clear to me at this stage where this should be resolved, so hopefully a discussion between us can resolve...

@samcarter would you have any insights into the question by John over here? https://github.com/jgm/pandoc/issues/5954#issuecomment-581502455 I think he's wondering whether the number relates to the specific offset of the pause. Would...