Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More useful printing for Pyreverse #8973

Closed
nickdrozd opened this issue Aug 23, 2023 · 0 comments · Fixed by #8974 or #9093
Closed

More useful printing for Pyreverse #8973

nickdrozd opened this issue Aug 23, 2023 · 0 comments · Fixed by #8974 or #9093
Labels
Minor 💅 Polishing pylint is always nice pyreverse Related to pyreverse component Work in progress
Milestone

Comments

@nickdrozd
Copy link
Contributor

Current problem

When Pyreverse runs, it prints out a bunch of "parsing" messages:

$ pyreverse pylint/pyreverse/
parsing pylint/pyreverse/__init__.py...
parsing /home/nick/pylint/pylint/pyreverse/inspector.py...
parsing /home/nick/pylint/pylint/pyreverse/plantuml_printer.py...
parsing /home/nick/pylint/pylint/pyreverse/printer.py...
parsing /home/nick/pylint/pylint/pyreverse/mermaidjs_printer.py...
parsing /home/nick/pylint/pylint/pyreverse/writer.py...
parsing /home/nick/pylint/pylint/pyreverse/__init__.py...
parsing /home/nick/pylint/pylint/pyreverse/dot_printer.py...
parsing /home/nick/pylint/pylint/pyreverse/main.py...
parsing /home/nick/pylint/pylint/pyreverse/printer_factory.py...
parsing /home/nick/pylint/pylint/pyreverse/diadefslib.py...
parsing /home/nick/pylint/pylint/pyreverse/utils.py...
parsing /home/nick/pylint/pylint/pyreverse/diagrams.py...

This is verbose and also not very useful. It's not even useful for debugging Pyreverse itself, since the parsing happens early on and errors generally happen later.

Desired solution

It would be cool if Pyreverse printed something more useful. Maybe some statistics or something?

Additional context

No response

@nickdrozd nickdrozd added pyreverse Related to pyreverse component Minor 💅 Polishing pylint is always nice labels Aug 23, 2023
@Pierre-Sassoulas Pierre-Sassoulas added the Needs specification 🔐 Accepted as a potential improvement, and needs to specify edge cases, message names, etc. label Aug 23, 2023
@Pierre-Sassoulas Pierre-Sassoulas added this to the 3.0.0a8 milestone Aug 25, 2023
@Pierre-Sassoulas Pierre-Sassoulas added Work in progress and removed Needs specification 🔐 Accepted as a potential improvement, and needs to specify edge cases, message names, etc. labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor 💅 Polishing pylint is always nice pyreverse Related to pyreverse component Work in progress
Projects
None yet
2 participants