Closed rsiera closed 7 years ago
Hey @alexeyklyukin, do you plan to merge @rsiera's PR? LMK, and I'll give it a Zappr thumbs-up.
@alexeyklyukin Did you have time to check this PR ? Maybe I'll create PR to different branch like phase_2 or something, because this one is blocking the next ones, which include test and adding psutil to the project.
@rsiera Apologizes, I was away and couldn't merge it. I will review it until Friday.
Thank for code-review ! I have applied all suggestions, except this one with setup_loggers, because I wasn't sure how to do that. If I understood correctly, the change was about to extract body of setup_loggers function to module definition, but how then can I pass options which contains required 'verbose' flag useful to configure logger ?
I think it makes sense to separate part of the logger that is configured based on command-line options from the rest of it. I've made a PR: https://github.com/rsiera/pg_view/pull/6
I also created another one to move global definitions around. It fixes the issue of not being able to toggle freeze or display of auxiliary processes. https://github.com/rsiera/pg_view/pull/7
Yes, my mistake. Both PR are merged, thanks.
👍
:+1:
@rsiera merged, thanks a lot for your efforts and perseverance.
Great collaboration and persistence, all :)
This is long-needed major rewrite that splits the single multi-thousand lines file into smaller modules. Adds use of psutil in the process.