You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@opennlp.apache.org by GitBox <gi...@apache.org> on 2022/12/04 09:59:48 UTC

[GitHub] [opennlp-site] rzo1 opened a new pull request, #74: OPENNLP-1402 - Replace Travis with GitHub actions on opennlp-site,

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

   # What does this PR do?
   
   - Replaces Travis CI with GitHub Actions to conduct a test build of the site, i.e. to validate PRs.
   - Adds some basic validation to ensure, that the docs are present and and `index.html` is generated.
   
   # Notes
   
   Travis isn't needed. Buildbot is building the site on a per-commit basis: https://ci2.apache.org/#/builders/41
   However, it is nice to have some sort of CI to ensure, that we can build pull requests and get a quick overview.
   
   Tested the workflow in my fork: https://github.com/rzo1/opennlp-site/actions/runs/3612720828/jobs/6088017976
   
   


-- 
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-site] jzonthemtn merged pull request #74: OPENNLP-1402 - Replace Travis with GitHub actions on opennlp-site,

Posted by GitBox <gi...@apache.org>.
jzonthemtn merged PR #74:
URL: https://github.com/apache/opennlp-site/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: dev-unsubscribe@opennlp.apache.org

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


[GitHub] [opennlp-site] rzo1 commented on a diff in pull request #74: OPENNLP-1402 - Replace Travis with GitHub actions on opennlp-site,

Posted by GitBox <gi...@apache.org>.
rzo1 commented on code in PR #74:
URL: https://github.com/apache/opennlp-site/pull/74#discussion_r1038950273


##########
README.md:
##########
@@ -20,7 +20,6 @@
 Welcome to OpenNLP Site Source Code
 ====================================
 
-[![Build Status](https://api.travis-ci.org/apache/opennlp-site.svg?branch=master)](https://travis-ci.org/apache/opennlp-site)

Review Comment:
   Good catch / idea. I added the GH badge + the missing license header for the gh actions file.



-- 
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-site] kinow commented on a diff in pull request #74: OPENNLP-1402 - Replace Travis with GitHub actions on opennlp-site,

Posted by GitBox <gi...@apache.org>.
kinow commented on code in PR #74:
URL: https://github.com/apache/opennlp-site/pull/74#discussion_r1038944508


##########
README.md:
##########
@@ -20,7 +20,6 @@
 Welcome to OpenNLP Site Source Code
 ====================================
 
-[![Build Status](https://api.travis-ci.org/apache/opennlp-site.svg?branch=master)](https://travis-ci.org/apache/opennlp-site)

Review Comment:
   Should we replace this by the GH action badgw? (Gine by me either way).
   
   Thanks!



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