Comment on rough edges of CI workflow
This commit is contained in:
parent
cd7a5a3954
commit
26ca403bec
1 changed files with 8 additions and 0 deletions
8
.github/workflows/ci.yml
vendored
8
.github/workflows/ci.yml
vendored
|
|
@ -69,6 +69,14 @@ jobs:
|
||||||
- name: Push changes
|
- name: Push changes
|
||||||
if: github.event_name == 'schedule' || ( github.event_name == 'workflow_dispatch' && inputs.updateFlakeLock )
|
if: github.event_name == 'schedule' || ( github.event_name == 'workflow_dispatch' && inputs.updateFlakeLock )
|
||||||
run: git push
|
run: git push
|
||||||
|
# `git push` only works because branch protection is not enabled.
|
||||||
|
#
|
||||||
|
# Currently branch protection is not effective anyway, since the only
|
||||||
|
# contributor (marienz) has admin permissions, and applying branch
|
||||||
|
# protection to administrators seems to be an "organization" feature.
|
||||||
|
#
|
||||||
|
# The supported path seems to be "create a PR and use the API to merge
|
||||||
|
# it", but that's more work to implement (see above): revisit later.
|
||||||
|
|
||||||
# TODO: try to improve caching.
|
# TODO: try to improve caching.
|
||||||
#
|
#
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue