Hello all,
UPDATE: The pipelines have run and all tests passed #336491916
Usually, the pipelines start running as soon as I push my code. But this
time they took longer to start and there was no sign of starting. This is my
first experience with pipelines so I assumed I messed up something from
my side.
Thanks,
Niteesh.
Why not?
This patch series introduces AQMP-TUI prototype. This prototype has been
helpfull in letting us try out different ideas and giving some insights
into things that we had to take care of in the upcoming TUI. It was also
helpfull in finding out bugs in the AQMP library.
The intent for this patch series is to get comments on the architectural
design of the prototype. These comments will lay down the foundation for
the upcoming TUI.
G S Niteesh Babu (6):
python: disable pylint errors for aqmp-tui
python: Add dependencies for AQMP TUI
python/aqmp-tui: Add AQMP TUI draft
python: add optional pygments dependency
python/aqmp-tui: add syntax highlighting
python: add entry point for aqmp-tui
python/Pipfile.lock | 20 ++
python/qemu/aqmp/aqmp_tui.py | 342 +++++++++++++++++++++++++++++++++++
python/setup.cfg | 36 +++-
3 files changed, 397 insertions(+), 1 deletion(-)
create mode 100644 python/qemu/aqmp/aqmp_tui.py
Thanks Niteesh, a few general comments that don't relate directly to the code:
1. It would be nice to be able to highlight/copy-paste things out of the history window, I seemingly can't right now.
2. It would be nice if the mouse scroll wheel worked on the history panel.
3. A greeting message like the old qmp-shell might be nice to see. It would be good if it explained how to quit the program (esc, ctrl^c) and send messages (alt+enter).
4. Some control hints or reminder text in the footer might be nice, for how to quit, send a message, etc.
For the next revision, I may ask you to start looking into making sure that mypy and pylint pass without exemptions. Do the best you can, and get as far as you are able. You can leave the warnings disabled for V3, but I'd like you to start taking a look now so that you know where the trouble spots are.
Thanks!
--js