conda-forge / conda-forge-ci-setup-feedstock

A conda-smithy repository for conda-forge-ci-setup.
BSD 3-Clause "New" or "Revised" License
13 stars 51 forks source link

fix unbound variable error with CPU_COUNT on GH Actions #192

Closed jaimergp closed 2 years ago

jaimergp commented 2 years ago

Checklist

https://github.com/conda-forge/conda-forge-ci-setup-feedstock/pull/183 introduced this bug. CPU_COUNT is unset on GH Actions, but we have a check later testing its value, which raises unbound variable due to set -u.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

jaimergp commented 2 years ago

@conda-forge-admin, please rerender

github-actions[bot] commented 2 years ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-ci-setup-feedstock/actions/runs/1881456673.

jaimergp commented 2 years ago

@conda-forge/core this is ready for review

hadim commented 2 years ago

Thanks! We also noticed this bug on our private conda forge that uses GH Actions.

hadim commented 2 years ago

I confirm the fix.