docker: Fix git clone path in Dockerfile for gdal-grass#5111
docker: Fix git clone path in Dockerfile for gdal-grass#5111echoix merged 1 commit intoOSGeo:mainfrom
Conversation
|
@nilason it seems better, but it can't find the Python grass package. |
|
It seems to me we need to run Docker in PR. |
In general we still have too much total CI time. With little more than occasional activity, the CIs get bogged down quickly and prevents further work. Which is a shame. |
Once the workflow itself is verified safe to run on PRs, then it would be possible to use a similar approach to the one used in additional checks, like for pre-commit. We could also add a workflow_dispatch trigger, that means we can manually launch a run. |
|
workflow_dispatch would perhaps be the least complicated solution. |
|
And that would also be available if there are other changes that are not in Dockerfile but potentially still affect. |
But, would be needed to run on the fork's branch, because I don't think we can select a branch to run on on a branch that comes from a forked repo (like all our PRs come from forks) |
For this, I only know that there's a way to do something that works with custom-purpose labels to control what to run/request what to run. But it's harder to setup, but afterwards it's kind of easy to use |
If my hypothesis in #5108 (comment) is right, this should solve the build failure