You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2020/09/30 10:59:55 UTC

[GitHub] [commons-text] kinow opened a new pull request #171: [TEXT-187] add graalvm dependencies in test scope

kinow opened a new pull request #171:
URL: https://github.com/apache/commons-text/pull/171


   Apache Jena did something similar a few days ago. But I used this post to refresh on the changes in JVM 15, and copied the dependencies (checked in search.maven.org too to confirm these are the latest versions): https://golb.hplar.ch/2020/04/java-javascript-engine.html


----------------------------------------------------------------
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.

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



[GitHub] [commons-text] garydgregory merged pull request #171: [TEXT-187] add graalvm dependencies in test scope

Posted by GitBox <gi...@apache.org>.
garydgregory merged pull request #171:
URL: https://github.com/apache/commons-text/pull/171


   


----------------------------------------------------------------
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.

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



[GitHub] [commons-text] garydgregory commented on pull request #171: [TEXT-187] add graalvm dependencies in test scope

Posted by GitBox <gi...@apache.org>.
garydgregory commented on pull request #171:
URL: https://github.com/apache/commons-text/pull/171#issuecomment-701381881


   Shouldn't we also test with Rhnio ? 
   https://search.maven.org/artifact/org.mozilla/rhino/1.7.13/jar ?


----------------------------------------------------------------
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.

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



[GitHub] [commons-text] coveralls commented on pull request #171: [TEXT-187] add graalvm dependencies in test scope

Posted by GitBox <gi...@apache.org>.
coveralls commented on pull request #171:
URL: https://github.com/apache/commons-text/pull/171#issuecomment-701333124


   
   [![Coverage Status](https://coveralls.io/builds/33827851/badge)](https://coveralls.io/builds/33827851)
   
   Coverage remained the same at 98.671% when pulling **b015004dd88d36732b939d6faae822ee1ea0f76e on kinow:TEXT-187** into **a526cc13eb8322da6d3980400cfdd8957c26366b on apache:master**.
   


----------------------------------------------------------------
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.

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



[GitHub] [commons-text] kinow commented on pull request #171: [TEXT-187] add graalvm dependencies in test scope

Posted by GitBox <gi...@apache.org>.
kinow commented on pull request #171:
URL: https://github.com/apache/commons-text/pull/171#issuecomment-701329512


   This solves Java 15 build. The Java 16 is broken, but it's experimental. Just needs someone else to review it, as I'm not using Java 15, not 100% there are no consequences in adding Graal VM as a test dependency.


----------------------------------------------------------------
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.

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