Skip to content
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

gno lint should provide warnings for exported global variables #1042

Open
moul opened this issue Aug 10, 2023 · 1 comment
Open

gno lint should provide warnings for exported global variables #1042

moul opened this issue Aug 10, 2023 · 1 comment
Labels
🗺️good first issue🗺️ Ideal for newcomer contributors

Comments

@moul
Copy link
Member

moul commented Aug 10, 2023

While there might be instances where allowing this behavior is justified, incorporating a mechanism to disable the warning could prove beneficial.

For instance, introducing a // nolint comment to selectively suppress the warning could enhance flexibility in code analysis.


This should warn:

var MyVar int

This shouldn't:

var MyVar int // nolint
@moul moul added the 🗺️good first issue🗺️ Ideal for newcomer contributors label Aug 10, 2023
@moul moul moved this to 🔵 Not Needed for Launch in 🚀 The Launch [DEPRECATED] Sep 5, 2023
@moul moul added this to the 💡Someday/Maybe milestone Sep 6, 2023
@moul moul moved this from 🔵 Not Needed for Launch to 🌟 Wanted for Launch in 🚀 The Launch [DEPRECATED] Sep 8, 2023
@moul
Copy link
Member Author

moul commented Sep 8, 2023

Should be extended with exported Const containing mutable fields like arrays.

@moul moul moved this to Todo in 💪 Bounties & Worx Dec 22, 2023
@wyhaines wyhaines added the hacktoberfest This might be a good issue for a hacktoberfest participant to handle. label Sep 30, 2024
@wyhaines wyhaines removed the hacktoberfest This might be a good issue for a hacktoberfest participant to handle. label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗺️good first issue🗺️ Ideal for newcomer contributors
Projects
Status: Todo
Status: 🌟 Wanted for Launch
Development

No branches or pull requests

3 participants