`make lint` gives extraneous feedback

[melinath]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
• Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
• If you are interested in working on this issue or have submitted a pull request, please leave a comment.
• If an issue is assigned to the modular-magician user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If an issue is assigned to a user, that user is claiming responsibility for the issue. If an issue is assigned to hashibot, a community member has claimed the issue already.

Currently, we encourage people opening PRs to run make lint to catch any formatting errors; however, make lint frequently catches "issues" on user's machines that are not considered a problem in ci/cd. For example, see GoogleCloudPlatform/magic-modules#4689 (comment) (or run make lint on your own machine.)

We should correct this mismatch so that the output from make lint is actionable for contributors.

[rileykarson]

Note: It seems to ignore the config file locally, and I have not been able to figure it out

[melinath]

Reopening as the fix (though it makes an improvement) doesn't fully resolve the issue - see comments on the PRs.

[ScottSuarez]

Reopening as the fix (though it makes an improvement) doesn't fully resolve the issue - see comments on the PRs.

Ah sorry about that, I removed the closes from one of the prs but not the other !

[melinath]

Reopening as the fix (though it makes an improvement) doesn't fully resolve the issue - see comments on the PRs.

Ah sorry about that, I removed the closes from one of the prs but not the other !

No worries! :-)