We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I recently faced some cases where it seemed relevant and more readable to use nested it / then / else in new recipe (rattler) format, instead of one-lining the expression. See here for a concrete case : https://github.com/conda-forge/eigenpy-feedstock/blob/5f141210cc57dc0c8cfbe9db4d7e59c3cd3f6347/recipe/recipe.yaml#L36-L48 It seems that rattler build supports such nested selectors, but conda smithy does not. Here a simpler reproducible example that fails with conda smithy linter:
it / then / else
rattler build
conda smithy
- if: arm64 then: - if: linux then: - foo else: - notfoo
I guess the first question raised here before going further is: should it be supported in the new recipe v2 standard ?
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Comment:
I recently faced some cases where it seemed relevant and more readable to use nested
it / then / else
in new recipe (rattler) format, instead of one-lining the expression. See here for a concrete case : https://github.com/conda-forge/eigenpy-feedstock/blob/5f141210cc57dc0c8cfbe9db4d7e59c3cd3f6347/recipe/recipe.yaml#L36-L48It seems that
rattler build
supports such nested selectors, butconda smithy
does not.Here a simpler reproducible example that fails with conda smithy linter:
I guess the first question raised here before going further is: should it be supported in the new recipe v2 standard ?
The text was updated successfully, but these errors were encountered: