Conversation
This PR adds support for generating v1 recipes using `conda-recipe-manager` to convert recipes generated with `conda-skeleton` to the newer format.
1fc6fe3 to
ff0ce08
Compare
beckermr
left a comment
There was a problem hiding this comment.
We should not merge this. Instead we should centralize around grayskull and move all recipe generation to that package.
|
This is coming from #5880. Initially the integration strategy used I don't see it as controversial, but if this is not going in, we should think of deprecating |
|
Yes, I think we should split out recipe generation in the same way we split out conda-index. |
I don't know much about the history of If anyone wants to give that a try, I would be very grateful and would happily review that PR. I would personally look at the |
|
Related issue #4460 |
|
If anyone wants to split Full files: Functions: |
To add to this thought, a # Or `RecipeParserDeps` if dependency functions are needed.
from conda_recipe_manager.parser.recipe_parser import RecipeParser
from pathlib import Path
parser = RecipeParser("")
parser.set_variable("version", "1.2.3")
parser.patch({"op": "add", "path": "/build", "value": {})
parser.patch({"op": "add", "path": "/build/number", "value": 0)
# Do this for all values you could want, branch on V0/V1 ...
Path("/path/to/recipe/file/meta.yaml").write_text(parser.render(), encoding="utf-8")Tracking this idea here: conda/conda-recipe-manager#482 |
This PR adds support for generating v1 recipes using
conda-recipe-managerto convert recipes generated withconda-skeletonto the newer format.Description
Checklist - did you ...
newsdirectory (using the template) for the next release's release notes?