search

cerfacs-globc / icclim

icclim: Python library for climate indices and climate indicators calculation.
https://icclim.readthedocs.io/en/latest/
Apache License 2.0
80 stars 32 forks source link

FIX: replace origin by dynamic value in CI pipeline #250

Closed bzah closed 1 year ago

bzah commented 1 year ago

Fix CI when running on a fork of icclim.

bzah commented 1 year ago

It looks like it's currently impossible to get the fork name from where the Pull request was requested in the github action workflow. So we can't checkout the branch and commit update automatically.

A workaround would be to trigger an specific workflow on push events on master after the PR merge, thus creating a new PR with the commits (logo update + API generation), but this is not satisfactory to me. See https://github.com/actions/checkout/issues/124#issuecomment-584485632

In the meantime, PR coming from forks should be doubled checked before merging and the checks can be ignored.