Skip to content

Conversation

Steffengreiner
Copy link
Contributor

@Steffengreiner Steffengreiner commented Dec 4, 2020

Purpose of this PR
This PR intends to update xml-manager-lib to the current Qube Template:

  1. Add Qube Template Files and remove outdated Travis files and Readme
  2. Removes Parent Pom and its dependencies from pom.xml and move its dependencies and versions to the xml-manager-lib pom.xml
  3. Replace outdated com.sun.xml.bind jaxb-core dependency with org.glassfish.jaxb jaxb-runtime dependency to account for the move from JDK 8 to JDK 11

@Steffengreiner Steffengreiner requested a review from a team December 4, 2020 16:09
@Override
void start(BundleContext context ) throws Exception {

System.out.println( context.getBundle().getSymbolicName() + ": Hello OSGi-World." )
Copy link
Member

Choose a reason for hiding this comment

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

are these and the car example classes needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They're introduced by the QUBE Template, I'm unsure how to proceed here since we would have to delete them every time we merge TEMPLATE back into development, (when changes are made to the scaffolding code)

Copy link
Member

@wow-such-code wow-such-code left a comment

Choose a reason for hiding this comment

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

It seems you solved the issue with Java 11? If so, please disregard the suggestions to return to Java 8.

@Steffengreiner Steffengreiner merged commit 964ab16 into development Dec 7, 2020
@Steffengreiner Steffengreiner deleted the merge_template branch December 7, 2020 09:54
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.

2 participants