-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ./... handling #4
Comments
I'm not able to reproduce this behavior - could you give an example? |
@mcandre Closing this for now, feel free to reopen if you're still running into this issue. |
Most Go tools support the standard syntax However, nakedret fails to support this. As a workaround, the user has to implement the recursion manually. Most Go users are not equipped to perform this reliably, and often end up breaking the portability of their build system. |
Hey there, thanks for the follow-up here! I unfortunately won't have bandwidth to address this in the immediate future, but I'm definitely open to PRs for supporting this 👍 |
Manually wrapping nakedret in Go code to recurse over project files. Here's an example: https://github.com/mcandre/mage-extras/blob/master/nakedret.go This code could be promoted into the nakedret tool itself in order to process the standard Go language ellipsis syntax faithfully. |
nakedret complains
warning: "./..." matched no packages
when there are multiple top-level*.go
source files to lint. Please correct nakedret's parsing of./...
and.\...
so that it includes top-level files, not just subdirectory paths.The text was updated successfully, but these errors were encountered: