BUG make sure to use only allowed dist names

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 53 forks source link

BUG make sure to use only allowed dist names #100

Closed beckermr closed 4 years ago

beckermr commented 4 years ago

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

conda-forge-linter commented 4 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.

beckermr commented 4 years ago

cc @isuruf

beckermr commented 4 years ago

@conda-forge-admin rerender

github-actions[bot] commented 4 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.

isuruf commented 4 years ago

This should only be done if there's test_downstreams text in recipe/meta.yaml. Saves a lot of time in CI and crucial for builds near the travis time limit.

beckermr commented 4 years ago

This version is not running the solver/finalize like we were doing before. It took ~20s total for my test job and ran no solves. Thus I am not worried about the performance.

isuruf commented 4 years ago

Great. Thanks.