[GitHub] [jena] afs commented on a change in pull request #682: JENA-1830: Put LTS in default travis-ci setup

[GitBox <gi...@apache.org>]

afs commented on a change in pull request #682: JENA-1830: Put LTS in default travis-ci setup
URL: https://github.com/apache/jena/pull/682#discussion_r371858480
 
 

 ##########
 File path: .travis.yml
 ##########
 @@ -4,9 +4,6 @@ install: true
 script: mvn -B clean install
 jdk:
   - openjdk8
-  - openjdk9
 
 Review comment:
   How do "allow_failures" get reported?  Does it still send out an email alert?
   
   If we had travis tied to, for example, project github actions, having a (non-failing) step for openjdk-ea makes sense. So if there is a preference and migration to use GH actions/Travis more (e.g. consolidating on GH for the sake of simplicity) then obviously this all needs revisiting in a new light.
   
   ATM The travis file is for contributors. For the project, we do have Jenkins builds that reports as well. I'm not sure whether "allow fail" makes a difference - not sure people would look!
   
   

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org