Skip to content

Conversation

@tbouffard
Copy link
Member

Extract interfaces to ensure that all methods are implemented in both BpmnElementsRegistry and the specific registries. This will also let reuse the method signatures in

  • CssClassesRegistry
  • ElementsRegistry
  • OverlaysRegistry
  • StyleRegistry

Add JSDoc existing category to interfaces and make links in "see" directive works (at least in the HTML report produced by typedoc)

closes #2803
covers #2813

Extract interfaces to ensure that all methods are implemented in both BpmnElementsRegistry and the specific registries.
This will also let reuse the method signatures in
  - CssRegistry
  - ElementsRegistry
  - OverlaysRegistry
  - StyleRegistry

JSDoc add existing category to interfaces and make links in "see" directive works (at least in the HTML report produced
by `typedoc`)
@tbouffard tbouffard added the refactoring Code refactoring label Aug 28, 2023
@github-actions
Copy link

github-actions bot commented Aug 28, 2023

♻️ PR Preview 9d6c617 has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@github-actions
Copy link

github-actions bot commented Aug 28, 2023

♻️ PR Preview 9d6c617 has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@tbouffard tbouffard merged commit 38a6e54 into master Aug 28, 2023
@tbouffard tbouffard deleted the refactor/registries_extract_interfaces branch August 28, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] Split the code of BpmnElementsRegistry and the code of its collaborators

3 participants