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

Generated mermaid files do not implement the --module-names option of pyreverse. #10164

Open
frank-pet opened this issue Jan 1, 2025 · 0 comments
Labels
Enhancement ✨ Improvement to a component pyreverse Related to pyreverse component

Comments

@frank-pet
Copy link

frank-pet commented Jan 1, 2025

Bug description

The mermaid files generated by the -ommd option do not implement the -m, --module-names option. An inspection of pyreverse/mermaidjs_printer.py shows that the module information indeed present and removed before outputting it to the mermaid file.

Note that -m[y|n] works fine for .dot files...

Configuration

pylint/pyreverse/mermaidjs_printer.py

Command used

pyreverse -A -my -f ALL -ommd filename.py

Pylint output

valid mermaid file with only the classnames in the output.

Expected behavior

Would expect to see "modulename.classname" in the mermaid file, vs just "classname".

Pylint version

pylint 2.16.2, but also in 3.3.0 at lines 60, 76, and 77.

OS / Environment

debian bookworm, arm64

@frank-pet frank-pet added the Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling label Jan 1, 2025
@Pierre-Sassoulas Pierre-Sassoulas added Enhancement ✨ Improvement to a component pyreverse Related to pyreverse component and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ Improvement to a component pyreverse Related to pyreverse component
Projects
None yet
Development

No branches or pull requests

2 participants