ci: Skip lint on forks #139

pull maflcko wants to merge 1 commits into bitcoin-core:main from maflcko:main changing 1 files +15 −18

maflcko commented at 12:13 PM on July 28, 2023: contributor

Currently lint may run on forks and eat up the capped Cirrus CI contingent. Fix this by just skipping it. It can be trivially enabled by forks, if they want to.

Also, select persistent workers based on type.
maflcko force-pushed on Jul 28, 2023

ci: Skip lint on forks

Also, select persistent workers based on type.

Also, remove unused greedy.

Also, transform normal fuzz task into persitent_worker task.

d8bd180041

maflcko force-pushed on Jul 29, 2023

in .cirrus.yml:18 in d8bd180041

  21 | -    greedy: true
  22 | -  use_compute_credits: true
  23 | -  only_if: $CIRRUS_BASE_BRANCH == 'main'
  24 | +  use_compute_credits: true  # https://cirrus-ci.org/pricing/#compute-credits
  25 | +  # https://cirrus-ci.org/guide/writing-tasks/#conditional-task-execution: Skip forks and non-pulls, to avoid needless costs
  26 | +  skip: $CIRRUS_REPO_OWNER != 'bitcoin-core' || $CIRRUS_PR == ""

hebasto commented at 8:42 AM on July 31, 2023:

Same as https://github.com/bitcoin-core/guix.sigs/pull/783#discussion_r1278965488.

dergoegge approved
fanquake merged this on Aug 15, 2023
fanquake closed this on Aug 15, 2023

Contributors

maflcko

hebasto

dergoegge