Skip to content

Comments

Origin/update depdencies#476

Open
RutujChheda wants to merge 1 commit intoqos-ch:masterfrom
RutujChheda:origin/update-depdencies
Open

Origin/update depdencies#476
RutujChheda wants to merge 1 commit intoqos-ch:masterfrom
RutujChheda:origin/update-depdencies

Conversation

@RutujChheda
Copy link

This pull request updates dependencies and build configurations to modernize the project and improve compatibility with newer Java versions and libraries. The most significant changes include updating Java versions in CI, upgrading OSGi and related dependencies, and modernizing the usage of Java collections.

Build and CI improvements:

  • Updated the GitHub Actions workflow to test against Java 21 and 25 in addition to existing versions, ensuring compatibility with the latest JDKs (.github/workflows/main.yml).
  • Added support for origin/* branch patterns in the workflow trigger to improve CI coverage (.github/workflows/main.yml).

Dependency upgrades:

  • Upgraded OSGi dependencies in osgi-over-slf4j/pom.xml to use org.osgi.core 6.0.0 and org.osgi.enterprise 5.0.0, providing support for newer OSGi features.
  • Updated org.apache.felix.main from 5.6.1 to 7.0.5 in integration/pom.xml for improved OSGi runtime support.
  • Upgraded javassist to version 3.30.2-GA and corrected its groupId in slf4j-ext/pom.xml.
  • Updated reload4j to 1.2.26 and junit to 4.13.2 in parent/pom.xml for improved stability and testing.

Code modernization:

  • Replaced usage of Properties with Hashtable<String, String> in Activator.java to align with OSGi best practices.
  • Minor formatting/comment update in Activator.java.

…r due to OSGi changes

Add JDK 21 and JDK 25 test runners

Signed-off-by: RutujChheda <rutujchheda@outlook.com>
@RutujChheda RutujChheda force-pushed the origin/update-depdencies branch from 700801f to ddafe78 Compare January 31, 2026 18:32
@RutujChheda
Copy link
Author

@ceki, could you please review this PR? The code should be easy to review and bump the version

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.

1 participant