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

[ENHANCEMENT] Quick fix could be more specific than disabling while robotcode diagnostics #309

Open
gohierf opened this issue Sep 16, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@gohierf
Copy link

gohierf commented Sep 16, 2024

Is your enhancement request related to a problem? Please describe.
Robotcode recently add more granularity to disable diagnostics rule by rule rather than the whole diagnostics.
Quick fix shortcut still suggest disabling the whole diagnostics with # robotcode: ignore.

Describe the solution you'd like
When using quick fix for Variable XXX is not used, quick fix shortcuts add the # robotcode: ignore comment.
I would prefer the quick fix to add # robotcode: ignore[variable-not-used].

Describe alternatives you've considered
Not using the quick fix shortcut.

Additional context
This enhancement would help to spread the word about the new feature.

@gohierf gohierf added the enhancement New feature or request label Sep 16, 2024
@d-biehl d-biehl self-assigned this Sep 16, 2024
@d-biehl
Copy link
Member

d-biehl commented Sep 16, 2024

Additional options to consider:

  • Multiple Quick Fix Suggestions: In addition to the existing # robotcode: ignore option, it would be helpful to provide other Quick Fix suggestions. For instance, allow the user to turn the diagnostic into a warning or a hint instead of ignoring it completely.

  • Scope of Quick Fix Application: It would be beneficial to allow the Quick Fix to be applied not just on a single line, but also for larger scopes like the entire block, the whole file, or even globally. For the latter, options could include applying the fix via the VSCode settings or adding the rule to the robot.toml configuration file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants