Skip to content

Conversation

@jamesfwood
Copy link
Collaborator

No description provided.

@jamesfwood jamesfwood requested a review from Copilot September 10, 2025 18:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades CWS to use Camunda Platform 7.23.0 Enterprise Edition (EE) along with Jakarta EE migration and ActiveMQ Artemis migration for Spring 6 compatibility. The upgrade modernizes the application's infrastructure while maintaining backward compatibility.

  • Upgraded from Camunda 7.20.0 to 7.23.0-ee with enterprise features
  • Migrated from javax.* to jakarta.* dependencies for Jakarta EE compliance
  • Replaced ActiveMQ 5.x with ActiveMQ Artemis for improved performance and Spring 6 compatibility

Reviewed Changes

Copilot reviewed 89 out of 92 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
pom.xml Updated Camunda version, migrated to Jakarta dependencies, replaced ActiveMQ with Artemis
utils.sh Updated version numbers for Camunda 7.23.0-ee and Tomcat 10.1.36
Multiple config files Updated servlet API versions and JAX-RS configurations for Jakarta EE
install/cws-ui/broker.xml New Artemis broker configuration replacing legacy ActiveMQ config
Java source files Updated imports from javax.* to jakarta.* across all modules
GitHub workflows Added Camunda EE credentials and license configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jamesfwood jamesfwood requested review from jdrodjpl and jl-0 September 16, 2025 22:01
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3 Security Hotspots
3.2% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link
Collaborator

@jdrodjpl jdrodjpl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, shouldn't affect any of the stuff I've done w/ UI or the various bug fixes I've done.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants