You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
QA: The code of qtop has already been through ruff (in a configuration which is now part of the git tree), however there are more things to check and aspects of the code waiting to be improved, without necessarily changing its functionality.
A project of its own is to try the qtop repository against different online tools and see what they report and how (and how easy are the fixes involved). At least of these should be adopted for a day-to-day usage.
Hello! How do i contribute to this? I have tried forking the repo and feeding it against deepsource. I see a lot of suggested fixes. How do i go forward now? Thank you!
@SumirSeth thanks for tackling this! Yes, I have seen the same with deepsource; different tools give different suggestions.
No need to fix them yet: let's just iterate the checks through the different tools (in separate repos, to do the comparison).
I have already seen that if you do one of them the others are quite similar (use different repo names or project names)
I propose to review them together, if would like to do that or, find some other procedure which is mutually convenient.
If you could share the URLs to review together here in this issue, that would be the next step.
QA: The code of qtop has already been through
ruff
(in a configuration which is now part of the git tree), however there are more things to check and aspects of the code waiting to be improved, without necessarily changing its functionality.A project of its own is to try the qtop repository against different online tools and see what they report and how (and how easy are the fixes involved). At least of these should be adopted for a day-to-day usage.
The way to go about it is:
The text was updated successfully, but these errors were encountered: