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.

Sign up for GitHub

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

Jump to bottom

Improvement/remove camunda from base and misc cleanup #295

Merged

essweine merged 4 commits into main from improvement/remove-camunda-from-base-and-misc-cleanup

Feb 16, 2023

Contributor

essweine commented Feb 16, 2023

This PR

removes references to the camunda namespace in the base parser
cleans up the camunda parser a bit (the namespace is now defined only in one place rather than half a dozen)
removes some additional unused tests and diagrams
deletes some unused, duplicative, and debugging methods from task and workflow that have been irritating me for a while

essweine added 4 commits

February 15, 2023 20:46


          remove camunda namespace from base bpmn parser

895b2cc


          rely on top level camunda parser for almost all namespace references

408bc67


          remove some other unused diagrams and tests

8f14d10


          remove a few unused, duplicative, and debugging methods

28b579b

jbirddog approved these changes

View reviewed changes

sonarcloud bot commented Feb 16, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

96.6% Coverage
0.0% Duplication

essweine merged commit 791f335 into main

essweine deleted the improvement/remove-camunda-from-base-and-misc-cleanup branch

February 16, 2023 15:26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet