You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wayang.apache.org by "glauesppen (via GitHub)" <gi...@apache.org> on 2023/02/24 11:18:21 UTC

[GitHub] [incubator-wayang-website] glauesppen opened a new pull request, #19: Travis migration to Github Actions

glauesppen opened a new pull request, #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19

   Hi all,
   
   
   I build a github actions file to migrate from Travis. This script validates the PR open to merge into main branch and the pushes into main branch. We can extend it to other branches if needed.
   
   There is a few questions I want to discuss with you all before we merge this PR:
   - I saw we are using a ./script/cibuild file to update the jekyll. What do you think about putting those commands directly in the actions script?
   - I inserted a release changelog plugin into the actions script. What do you think about using it to log our releases in prod? Here is the docs: https://github.com/marketplace/actions/release-changelog-builder
   
   Let me know your thoughts :)


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] glauesppen commented on pull request #19: Travis migration to Github Actions

Posted by "glauesppen (via GitHub)" <gi...@apache.org>.
glauesppen commented on PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19#issuecomment-1447242989

   I am changing the approach to the release creator, since apache only allows verified github actions on the marketplace


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] JorgeQuiane commented on pull request #19: Travis migration to Github Actions

Posted by "JorgeQuiane (via GitHub)" <gi...@apache.org>.
JorgeQuiane commented on PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19#issuecomment-1443764031

   Hi all,
   
   - I don't see any issue in having cibuild's commands in the action script, so I vote for doing so. This would be more transparent for everyone IMO.
   - +1 for also having the release changelog in the action script :)
   
   Best!
   


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] 2pk03 merged pull request #19: Travis migration to Github Actions

Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 merged PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] 2pk03 commented on pull request #19: Travis migration to Github Actions

Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19#issuecomment-1455700756

   Thank you, I'm going and merge this PR! Congratulations on your first commit to Wayang!


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] 2pk03 commented on pull request #19: Travis migration to Github Actions

Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19#issuecomment-1443765603

   +1, if no objections in the next 48h I'm going to merge. 


-- 
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@wayang.apache.org

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

[GitHub] [incubator-wayang-website] 2pk03 commented on pull request #19: Travis migration to Github Actions

Posted by "2pk03 (via GitHub)" <gi...@apache.org>.
2pk03 commented on PR #19:
URL: https://github.com/apache/incubator-wayang-website/pull/19#issuecomment-1447794134

   Sure, than we wait with the merge. 


-- 
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@wayang.apache.org

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