Skip to content

feat: Allow $schema top-level field in recipe.yaml - #2645

Merged
wolfv merged 2 commits into
prefix-dev:mainfrom
bollwyvl:patch-2
Jul 28, 2026
Merged

feat: Allow $schema top-level field in recipe.yaml#2645
wolfv merged 2 commits into
prefix-dev:mainfrom
bollwyvl:patch-2

Conversation

@bollwyvl

@bollwyvl bollwyvl commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

References

Changes

  • accept top-level $schema, otherwise not used
  • replace magic comment with $schema in a test file (presumably it gets parsed)

Motivation

$schema allows for explicitly describing the contract a file meets without magic comments.

This convention is now supported by most third-party tools, most recently yaml-language-server 1.24.0.

Future Work

@bollwyvl bollwyvl changed the title Allow $schema top-level field in recipe.yaml feat: Allow $schema top-level field in recipe.yaml Jul 9, 2026
format!("unknown top-level field '{}'", key_str),
*key.span(),
)
.with_suggestion("valid top-level fields are: package, build, about, requirements, extra, source, tests, schema_version, context"));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you amend this as well?

@wolfv

wolfv commented Jul 28, 2026

Copy link
Copy Markdown
Member

Can you also make a CEP amendment?

@wolfv
wolfv merged commit f3cfbd7 into prefix-dev:main Jul 28, 2026
13 checks passed
@wolfv wolfv mentioned this pull request Jul 28, 2026
@bollwyvl

Copy link
Copy Markdown
Contributor Author

CEP amendment

Is that gated by a CLA?

@bollwyvl

Copy link
Copy Markdown
Contributor Author

also, thanks!

@bollwyvl
bollwyvl deleted the patch-2 branch July 28, 2026 16:31
@wolfv

wolfv commented Jul 28, 2026

Copy link
Copy Markdown
Member

Is that gated by a CLA?

Pretty sure it is not!

@bollwyvl

Copy link
Copy Markdown
Contributor Author

CEP amendment

Yep, no lawyers were harmed in the creation of conda/ceps#186

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants