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.

Sign up for GitHub

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

Jump to bottom

Unused variable or param should offer a code action to remove #5613

Closed
majastrz opened this issue Jan 12, 2022 · 3 comments
Closed

Unused variable or param should offer a code action to remove #5613

majastrz opened this issue Jan 12, 2022 · 3 comments
Assignees
@nilshedstrom
Labels
devdiv Related to Bicep tooling efforts in DevDiv enhancement New feature or request story: linter
Milestone
v0.7

Comments

@majastrz
Copy link
Member

We the linter finds an unused param or var, the list of code fixes only offers an option to disable the warning. We should also offer an option to remove the unused symbol.

image
@majastrz majastrz added the enhancement New feature or request label Jan 12, 2022
@ghost ghost added the Needs: Triage 🔍 label Jan 12, 2022
@majastrz majastrz changed the title Unused variable or param should offer code action to remove Unused variable or param should offer a code action to remove Jan 12, 2022
@alex-frankel alex-frankel added this to the Committed Backlog milestone Jan 12, 2022
@nilshedstrom
Copy link
Contributor

I can implement this.
But before I start I would like someone to look at my previous code action PR #5544

@StephenWeatherford StephenWeatherford added the devdiv Related to Bicep tooling efforts in DevDiv label Jan 21, 2022
@nilshedstrom
Copy link
Contributor

@StephenWeatherford What does the devdiv label mean?
Is it ok if I start working on this?

@StephenWeatherford
Copy link
Contributor

Go for it, thanks!

@nilshedstrom nilshedstrom mentioned this issue Jan 29, 2022
Merged
@StephenWeatherford StephenWeatherford moved this to Todo in Bicep May 31, 2022
@StephenWeatherford StephenWeatherford moved this from Todo to Done in Bicep May 31, 2022
@ghost ghost locked as resolved and limited conversation to collaborators May 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nilshedstrom nilshedstrom

Labels
devdiv Related to Bicep tooling efforts in DevDiv enhancement New feature or request story: linter
Projects
Bicep
Archived in project
Milestone
v0.7
Development

No branches or pull requests
4 participants
@nilshedstrom @StephenWeatherford @alex-frankel @majastrz