search

UWPCE-PythonCert / PythonCertDevel

Development Repo for the Certificate Program
https://uwpce-pythoncert.github.io/PythonCertDevel/index.html
Other
7 stars 15 forks source link

Try Travis CI/CD (automate build_gh_pages.sh) #175

Closed hosungs closed 5 years ago

hosungs commented 5 years ago

This is to automate the HTML building (build_gh_pages.sh can be retired with this).

See this for a Travis CI/CD build result on my fork.

See this for the automatically deployed result.

See this commit that was auto-committed by the Travis CI/CD job.

To enable CI/CD on UWPCE-PythonCert/PythonCertDevel, an admin (which I'm not) needs to sign up at https://travis-ci.org/, enable the repo for CI, generate a personal access token (public_repo scope only), set the environment variable GITHUB_TOKEN with the generated personal access token at the Travis CI job settings page, and merge this PR.

I'd be happy to do that if I'm given the admin right.

Thanks, Hosung

hosungs commented 5 years ago

Personal access token-related info can be found here.

PythonCHB commented 5 years ago

@hosungs: Thanks for doing this!

@natasha-aleksandrova and I had talked about it, but never got it going.

I'll give you admin rights, but we want to be careful about our workflow once this is set up. I haven't looked at the code yet, but it would probably only deploy the master branch, and then se can pupsh changes to other branches, and test the build there, before merging into master.

This will make things easier.

PythonCHB commented 5 years ago

@hosungs: you are now an "owner", let's see if that works -- gitHub seems to have changed its permission structure, so it's not clear how to give someone the necessary permissions.

hosungs commented 5 years ago

Thanks Chris for giving me the privilege. Configured Travis CI for this repo and merged this PR. See the first CI/CD result here, and the resulting commit to gh-pages branch. No more need to run build_gh_pages.sh.

hosungs commented 5 years ago

Just wanted to mention there are build warnings as follows. I haven't looked at them in detail, but I suppose they might be just easy things to fix for you...?

https://travis-ci.org/UWPCE-PythonCert/PythonCertDevel/jobs/447449674#L722-L723 https://travis-ci.org/UWPCE-PythonCert/PythonCertDevel/jobs/447449674#L921-L922

hosungs commented 5 years ago

OK, the above build warnings are now all fixed in this build by the commit 82a3e12.