Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

:green_heart: Let's move from Travis to GitHubAction. #26

Open
tkoyama010 opened this issue Apr 4, 2020 · 4 comments
Open

:green_heart: Let's move from Travis to GitHubAction. #26

tkoyama010 opened this issue Apr 4, 2020 · 4 comments

Comments

@tkoyama010
Copy link

@tkoyama010 tkoyama010 commented Apr 4, 2020

Hi all. How about moving CI from Travis to GitHubActions? Let me know what you think.

@tkoyama010
Copy link
Author

@tkoyama010 tkoyama010 commented Apr 4, 2020

FYI this repository is good. This yml may be helpful.

@cocoatomo
Copy link
Collaborator

@cocoatomo cocoatomo commented Apr 4, 2020

Requirements of CI build is described in the wiki page.
Besides, I have a plan of PR build to display documentation built from source files that PR merged.

Please help me with an assessment.

GitHub Action

Pros.

  • Easily recognize types of event
  • Many GitHub Actions already exist are available
  • automatic GITHUB_TOKEN creation and expiration
  • ?

Cons.

Unknown.

  • Can GitHub Action fully satisfy the requirements above?
  • Is storing secrets easier than Travis ci?
  • Can access across repositories easier?
  • ?
@cocoatomo
Copy link
Collaborator

@cocoatomo cocoatomo commented May 9, 2020

The feature of automatic GITHUB_TOKEN creation is very helpful to push the results of PR builds. Moreover, the tokens have existence time limits, 60 minutes.

@cocoatomo
Copy link
Collaborator

@cocoatomo cocoatomo commented May 17, 2020

Two workflows, pr_build.yml and delete_pr_built_doc.yml, are now working!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.