-
Notifications
You must be signed in to change notification settings - Fork 30
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
Request: offer UI for the plugin, just like what we have on the IDE #40
Comments
@AndroidDeveloperLB I've used that option recently and it's a shitshow, it removes several things that are being used. Also be careful, it doesn't run on all modules. Say you have This plugin needs a bit of polishing but it gave me ridiculously superior result - only 3 false removals, easy to undo by hand (and configure build.gradle.kts to never touch them again). Compared to Android Studio where I got easily more than 50 false removals and is not configurable. Not to mention I think this plugin found way more things that should be removed (600 lines removed over 50 files today), though not certain since it's been a week since I ran the Android Studio command. |
@fmatosqg I know it sometimes has annoying results. No idea how I can set an exclusion list either (to avoid showing me results that I know that shouldn't be removed). Anyway, the request here is to have a nice UI like what we have today. Please try to make it have even better UI. For me, for example, I think it could be nice to tell it what to ignore (and show this list too). |
Hey,
I never contributed to this plugin though I found it useful and wanted to
share my opinion.
In my case a UI is not desirable, since I prefer to run it on command line
and see the progress. It takes over 20 minutes in my project.
The documentation shows how to skip certain folders and I believe it also
supports reg ex.
…On Thu, 4 Nov 2021, 20:52 AndroidDeveloperLB, ***@***.***> wrote:
@fmatosqg <https://github.com/fmatosqg> I know it sometimes has annoying
results. No idea how I can set an exclusion list either (to avoid showing
me results that I know that shouldn't be removed).
Each time I use this tool, I use it very carefully, checking things that
seem like a suspicious false-positive.
Anyway, the request here is to have a nice UI like what we have today.
This way there could be a nice competition and a good alternative to what
we have today.
Please try to make it have even better UI. For me, for example, I think it
could be nice to tell it what to ignore (and show this list too).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABI4Y4LE6A2K2XDZSYLLSADUKJJW5ANCNFSM5F6DJ53Q>
.
|
@fmatosqg The request is not to remove the capability of using it in a command line. |
Like what we get from this :
Come to think of it, what are the advantages of using this plugin over this ?
The text was updated successfully, but these errors were encountered: