On Thu, Aug 19, 2021 at 1:39 PM G S Niteesh Babu <niteesh...@gmail.com> wrote:
> Hello all, > > Gitlab: https://gitlab.com/niteesh.gs/qemu/-/commits/aqmp-tui-prototype-v4 > CI: https://gitlab.com/niteesh.gs/qemu/-/pipelines/356024270 > > Revision since v3: > 1) Added docstrings > 2) Minor changes in AQMP TUI Draft > 3) Switched to constant retry delay in QMP connection manager and other > minor changes. > > G S Niteesh Babu (7): > python: disable pylint errors for aqmp-tui > python: Add dependencies for AQMP TUI > python/aqmp-tui: Add AQMP TUI draft > python: Add entry point for aqmp-tui > python: add optional pygments dependency > python/aqmp-tui: Add syntax highlighting > python/aqmp-tui: Add QMP connection manager > > python/Pipfile.lock | 20 + > python/qemu/aqmp/aqmp_tui.py | 681 +++++++++++++++++++++++++++++++++++ > python/setup.cfg | 29 +- > 3 files changed, 729 insertions(+), 1 deletion(-) > create mode 100644 python/qemu/aqmp/aqmp_tui.py > > -- > 2.17.1 > > Hiya, please specify what your branch is based on when you submit your patchsets using the Based-On tag. - It's time to drop patch #1 entirely. - We can drop the word 'draft' from the commit messages now, let's try to make this solid. - If you want to keep the TODO comments in the code, add a pylint configuration item that allows "todo" but disallows "fixme" and "xxx" comments. Detailed review to follow tomorrow, but you can start working on these items right away. I might also squash patch #7 directly into patch #3, but haven't looked too closely yet. (Note that due to an update to pylint that happened just today, check-tox is going to fail now -- that's fine, it's not your fault. As long as check-pipenv works, you're in good shape. I'll have a fix in my python branch tomorrow for these problems.)
