-
Notifications
You must be signed in to change notification settings - Fork 23
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
Switch to GitHub Actions #308
Conversation
I tried to update the OpenFisca-Core doc update trigger, but encounter issues with GitHub access tokens. I followed the Actions documentation on
Here is the token I created: |
I got the trigger to run from a Core commit!! The |
Thanks a lot for your reviews @Ndpnt! Shipping this now 🤘 |
Set up dependencies
Increase readability
Co-authored-by: Nicolas Dupont <[email protected]>
Rename Test workflow to Validate as linting is not exactly a test Rename Build job to include critical caching part Co-authored-by: Nicolas Dupont <[email protected]>
Make external dependencies more explicit on top Co-authored-by: Nicolas Dupont <[email protected]>
Co-authored-by: Nicolas Dupont <[email protected]>
Co-authored-by: Nicolas Dupont <[email protected]>
Deploy trigger updated in openfisca/openfisca-core#1211, all done! ✅ |
The documentation now publishes directly to the `gh-pages` branch See <openfisca/openfisca-doc#308>
The documentation now publishes directly to the `gh-pages` branch See <openfisca/openfisca-doc#308>
Details in #307.
Fixes #307, #284.
Integrates #286.
If merged:
test-cd
branch in openfisca/openfisca.org should be erased.cd
branch in this repository should be erased.Proofs that this works, that have to be double-checked:
test-cd
branch that was created as a test target branch as a duplicate ofgh-pages
. All files are changed because of the Sphinx (static website generator) update, but only those under thedoc
folder, as they should.