Matthew R. Becker
Matthew R. Becker
IDK which tool is breaking things here, but we'll need to investigate and fix.
Right yes thanks for the details everyone. @trim21 did you try rerendering the feedstock? I thought this [PR](https://github.com/conda-forge/conda-smithy/pull/2314/files) from @isuruf should have forced smithy to use booleans correctly?
> then conda-smithy render ci config, it use a str "true" or "false" as is_python_min so only this work: skip: "is_python_min != 'true'" So in the ci support file, it...
I don't see a link to the feedstock in the chat. That would be helpful if you have it.
So the type in the pinning file shouldn't matter and I believe that conda-build does what it does. The PR from @isuruf should have coerced "true" values to actual booleans...
> Which chat? > > If you want a reproduce I can make one for you Ah sorry for not being clear. I meant the comments on this issue. Presumably...
Ahhh thank you! Let's wait for @isuruf to chime in on what he thinks is the best path forward.
I'm happy to have that pr merged but someone from prefix will need to do it.
I think we want to increase the lower bound for rattler-build-conda-compat in our envs. I have made a PR to smithy's feedstock and will submit PRs here and to the...
- [x] PR to feedstock: https://github.com/conda-forge/conda-smithy-feedstock/pull/358 - [x] PR to smithy repo: https://github.com/conda-forge/conda-smithy/pull/2375 - [x] PR to repodata patches: https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/1069 - [x] issue on conda-build to make the change there...