Skip to content
New issue

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

schema_version: 1 fails linter #2201

Closed
1 task done
lucascolley opened this issue Jan 2, 2025 · 2 comments
Closed
1 task done

schema_version: 1 fails linter #2201

lucascolley opened this issue Jan 2, 2025 · 2 comments
Labels

Comments

@lucascolley
Copy link
Member

Solution to issue cannot be found in the documentation.

  • I checked the documentation.

Issue

I reported this at conda-incubator/conda-recipe-manager#280 (comment), but @schuylermartin45 informed me that the linter shouldn't be failing here. cc @nichmor @wolfv

I needed the following diff for the linter to pass after using crm convert:

diff --git a/recipe/recipe.yaml b/recipe/recipe.yaml
index 372e4c3..26fe8d2 100644
--- a/recipe/recipe.yaml
+++ b/recipe/recipe.yaml
@@ -1,5 +1,3 @@
-schema_version: 1
-
 context:
   name: array-api-extra
   version: 0.4.0
@@ -56,4 +54,3 @@ about:
 extra:
   recipe-maintainers:
     - lucascolley
-

Installed packages

.

Environment info

.
@schuylermartin45
Copy link

Specifically: schema_version is a new field introduced in CEP-13 with the V1 recipe format that the linter must be okay with if it is to support the new format.

@mgorny
Copy link
Contributor

mgorny commented Jan 6, 2025

I'm going to try tackling this.

mgorny added a commit to mgorny/conda-smithy that referenced this issue Jan 6, 2025
Allow the v1 recipes to contain a top-level `schema_version` key,
and expect it to be ordered first.  The key is defined in CEP 14:
https://conda.org/learn/ceps/cep-0014#schema-version

Fixes conda-forge#2201
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants