-
Notifications
You must be signed in to change notification settings - Fork 152
fix: Revert "fix: ignore go.mod by default" #1365
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
Conversation
This reverts commit 4eeb460. Original PR: crate-ci#684 Original Issue: crate-ci#683 This didn't cause `go.mod` to be ignoredinstead it caused it to not be counted as go code. However, `go.mod` and `go.work` are similar file types and both are more of package specifications than lock files or other machine-only content. See crate-ci#1363
Pull Request Test Coverage Report for Build 17303202645Details
💛 - Coveralls |
|
Nitpicking: I would fix the typo in commit message
|
|
Is there a need to add |
Those are in the lockfile type which is not checked |
|
This change equivalent to add locally add go.mod and go.work to extends glob for type go, right? Because when I try it locally it reports typos for go.mod files Try this one https://github.com/noborus/ov/blob/master/go.mod Maybe I'm wrong, but I prefer to ask. |
|
Yes, this makes |
Indeed. I didn't see it.
Then, it reports errors. I feel like in .go files everything is checked, and not only comments. But we expect something different for go.mod and go.work where we only need to check comment. Please try with the go.mod file I provided |
|
Oh OK. Then I would exclude go.mod and go.work totally. That's what I did for many projects. |
|
I just saw your comment here.
You are right. Let's continue the discussion in #1363 |
This reverts commit 4eeb460.
Original PR: #684
Original Issue: #683
This didn't cause
go.modto be ignored instead it caused it to not be counted as go code.However,
go.modandgo.workare similar file types and both are more of package specifications than lock files or other machine-only content.See #1363