You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@opennlp.apache.org by "mawiesne (via GitHub)" <gi...@apache.org> on 2023/01/26 13:55:30 UTC

[GitHub] [opennlp-sandbox] mawiesne opened a new pull request, #74: Introduce-common-maven-module-structure-for-corpus-server-components

mawiesne opened a new pull request, #74:
URL: https://github.com/apache/opennlp-sandbox/pull/74

   - adjusts parent project (org.apache.apache) to version 18
   - adjusts Java language level to 11
   - introduces parent pom module `corpus-server` and reorganizes children: `-core, -impl, -connector, -tools`
   - updates several dependencies to more up-to-date versions to mitigate several CVEs
   - adjusts some array declarations to comply with Java, not C style
   - fixes existing, commented JUnit tests in `-connector`, yet they won't fully work -> Ignored for this PR


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [opennlp-sandbox] mawiesne merged pull request #74: Introduce common Maven module structure for corpus-server components

Posted by "mawiesne (via GitHub)" <gi...@apache.org>.
mawiesne merged PR #74:
URL: https://github.com/apache/opennlp-sandbox/pull/74


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: devnull-unsubscribe@infra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [opennlp-sandbox] rzo1 commented on pull request #74: Introduce-common-maven-module-structure-for-corpus-server-components

Posted by "rzo1 (via GitHub)" <gi...@apache.org>.
rzo1 commented on PR #74:
URL: https://github.com/apache/opennlp-sandbox/pull/74#issuecomment-1405076091

   Thanks for the changes. It is a good way to make the sandbox be maintainable / buildable. I guess, that `lucas` isn't maintained anymore and most likely not compatible with `3.x.x`, so it needs some love in the future. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org