search

conda-forge / conda-smithy

The tool for managing conda-forge feedstocks.
https://conda-forge.org/
BSD 3-Clause "New" or "Revised" License
146 stars 170 forks source link

Rerenders: do not raise on validation errors #1885

Closed jaimergp closed 4 months ago

jaimergp commented 4 months ago

Checklist

Closes #1879

For now it doesn't offer any configuration options, and just replaces the previous errors with warnings formatted like this:

WARNING  conda_smithy.configure_feedstock:configure_feedstock.py:2002 conda-forge.yml: $.noarch_platforms = ['eniac', 'zx80'] -> ['eniac', 'zx80'] is not valid under any of the given schemas

Let me know if you want to have an option in the CLI, but for now users could just run recipe-lint before or after 🤷

ytausch commented 4 months ago

Please don't use logger.warn, as it's deprecated. Your IDE should be configured to display warnings like these.

1865 will change it to warning.