Closed westinhuang closed 6 days ago
Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipes/check_shapes
) and found some lint.
Here's what I've got...
For recipes/check_shapes:
requirements: run: lark >= 1.1.0
should not contain a space between relational operator and the version, i.e. lark >=1.1.0
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 (recipes/check_shapes
) and found it was in an excellent condition.
Depends on PR: https://github.com/conda-forge/staged-recipes/pull/26765
@conda-forge-admin, please restart ci
Yeah think it was still mirroring to CDN
@conda-forge-admin , please restart CI
Looks ok to me. Let's ask for a 2nd opinion
@conda-forge/staged-recipes could you please review? 🙂
To help direct your pull request to the best reviewers, please mention a topic-specifc team if your recipe matches any of the following: conda-forge/help-c-cpp, conda-forge/help-cdts, conda-forge/help-go, conda-forge/help-java, conda-forge/help-julia, conda-forge/help-nodejs, conda-forge/help-perl, conda-forge/help-python, conda-forge/help-python-c, conda-forge/help-r, conda-forge/help-ruby,or conda-forge/help-rust. Thanks!
@conda-forge/help-python
@ocefpaf It keeps fail.
@ocefpaf It keeps fail.
Yeah. That pattern for multiple outputs is a bit finicky, I used a new one that should be easier to maintain and read.
Like the new pattern 🤩
Out of curiosity, does this work better with Grayskull?
Out of curiosity, does this work better with Grayskull?
I don't know. Sadly, Marcello is changing jobs and won't be working on grayskull anytime soon. I'd love to onboard more people to work on recipe updates, multiple outputs, etc. Maybe we should try outreachy again...
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).Fixes https://github.com/conda-forge/staged-recipes/issues/26766