update changelog

haidaraM · May 25, 2023 · ab9990a · ab9990a
1 parent 76253cb
commit ab9990a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,9 +5,7 @@
 - 🚀🚀Add support for MermaidJS 🚀🚀. See https://github.com/haidaraM/ansible-playbook-grapher/issues/137
 - Update various Dependencies: pytest, pytest-cov, ansible-core, pyquery etc...
 - ci: Add dependabot for github-actions
-- When using `--group-roles-by-name`, the roles used multiple times will include a counter corresponding to the number
-  of times they are used.
-- Rename some tests files
+- Rename some tests files..
 
 ## Breaking changes
 

diff --git a/ansibleplaybookgrapher/renderer/graphviz/__init__.py b/ansibleplaybookgrapher/renderer/graphviz/__init__.py
@@ -248,20 +248,18 @@ def build_role(self, role_node: RoleNode, color: str, fontcolor: str, **kwargs):
             url = self.get_node_url(role_node, "folder")
 
         plays_using_this_role = self.roles_usage[role_node]
-        name_suffix = ""
         if len(plays_using_this_role) > 1:
             # If the role is used in multiple plays, we take black as the default color
             role_color = "black"
             fontcolor = "#ffffff"
-            name_suffix = f"({len(plays_using_this_role)})"
         else:
             role_color, fontcolor = list(plays_using_this_role)[0].colors
 
         with digraph.subgraph(name=role_node.name, node_attr={}) as role_subgraph:
             role_subgraph.node(
                 role_node.id,
                 id=role_node.id,
-                label=f"[role] {role_node.name} {name_suffix}".strip(),
+                label=f"[role] {role_node.name}",
                 style="filled",
                 tooltip=role_node.name,
                 fontcolor=fontcolor,

diff --git a/ansibleplaybookgrapher/renderer/mermaid.py b/ansibleplaybookgrapher/renderer/mermaid.py
@@ -220,11 +220,9 @@ def build_role(self, role_node: RoleNode, color: str, fontcolor: str, **kwargs):
 
         plays_using_this_role = len(self.roles_usage[role_node])
         node_color = color
-        name_suffix = ""
         if plays_using_this_role > 1:
             # If the role is used in multiple plays, we take black as the default color
             node_color = "#000000"  # black
-            name_suffix = f"({plays_using_this_role})"
 
         # From parent to role
         self.add_link(
@@ -241,7 +239,7 @@ def build_role(self, role_node: RoleNode, color: str, fontcolor: str, **kwargs):
         self.roles_built.add(role_node)
 
         # Role node
-        self.add_text(f'{role_node.id}("[role] {role_node.name} {name_suffix}")')
+        self.add_text(f'{role_node.id}("[role] {role_node.name}")')
         self.add_text(
             f"style {role_node.id} fill:{node_color},color:{fontcolor},stroke:{node_color}"
         )