|
From: | Niteesh G. S. |
Subject: | Re: [PATCH v4 0/7] AQMP TUI Draft |
Date: | Sun, 22 Aug 2021 02:50:31 +0530 |
On Thu, Aug 19, 2021 at 1:39 PM G S Niteesh Babu <niteesh.gs@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.)
[Prev in Thread] | Current Thread | [Next in Thread] |