Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Implement quick fixes for Rubocop #7

Merged
merged 10 commits into from
Aug 23, 2022

Conversation

Verseth
Copy link

@Verseth Verseth commented Aug 21, 2022

Hi, I implemented quick fixes for Rubocop warnings.

My team has been longing for a feature like this for quite some time.
So I'd decided to try and implement it myself.

And if I may say so myself, it came together pretty nicely.

Auto-correctable cop:

Rubocop Quick Fix Tooltip - Correctable

Rubocop Quick Fix List - Correctable

Uncorrectable cop:

Rubocop Quick Fix Tooltip - Uncorrectable

Rubocop Quick Fix List - Uncorrectable

Here's a demo:

Rubocop Quick Fixes Demo

@garbusbeach
Copy link

OMG, waiting for it whole life! 🔥

@cub8
Copy link

cub8 commented Aug 23, 2022

Finally we will have easy quick fixes, kudos to you 👏

@faramund
Copy link

@Verseth great feature!

@LoranKloeze
Copy link
Owner

Very nice! Will check it and release it in a new version somewhere this week I hope.

@LoranKloeze LoranKloeze merged commit 7109905 into LoranKloeze:master Aug 23, 2022
@LoranKloeze
Copy link
Owner

New release with this functionality is now published to the marketplace in version 0.9.3. Thank you very much for this PR!

@ukrmaxim
Copy link

ukrmaxim commented Feb 2, 2023

screenshot
Need help. Indicates that there are no quick fixes available. Reinstalled several times. Ubuntu 22.04. VSCode 1.74.3
rubocop gem installed.

@Verseth
Copy link
Author

Verseth commented Feb 2, 2023

@ukrmaxim Looks like you've either got an old version of this extension or you've installed a different extension with similar functionality. The output has a different format.

image

This is the official marketplace page of this extension: https://marketplace.visualstudio.com/items?itemName=LoranKloeze.ruby-rubocop-revived

@ukrmaxim
Copy link

ukrmaxim commented Feb 3, 2023

@Verseth thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants