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

chore: enable unused-parameter from revive #1783

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmorel-35
Copy link
Contributor

@mmorel-35 mmorel-35 commented Jan 30, 2025

Description

unused-parameter: This rule warns on unused parameters. Functions or methods with unused parameters can be a symptom of an unfinished refactoring or a bug.

@mmorel-35
Copy link
Contributor Author

@shirou ,
I just rebase and fixed MacOs and Windows platforms
I'm wondering if it would be pertinent to cover every GOOS and GOARCH platforms with golangci-lint by the way

@mmorel-35 mmorel-35 marked this pull request as draft February 21, 2025 08:38
@mmorel-35
Copy link
Contributor Author

I have opened #1797 , I believ it's better to merge it first and then update this one.

@mmorel-35 mmorel-35 force-pushed the revive/unused-parameter branch 8 times, most recently from ca659f4 to 35e8062 Compare February 21, 2025 10:03
@mmorel-35 mmorel-35 force-pushed the revive/unused-parameter branch from 35e8062 to 37dd0eb Compare February 22, 2025 13:40
@mmorel-35 mmorel-35 marked this pull request as ready for review February 22, 2025 13:41
@mmorel-35 mmorel-35 force-pushed the revive/unused-parameter branch 4 times, most recently from a96c3b6 to 103801b Compare February 23, 2025 10:08
@mmorel-35 mmorel-35 force-pushed the revive/unused-parameter branch from 103801b to 1f43ea8 Compare February 25, 2025 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant