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

Allow $schema in eas.json #2624

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

saiichihashimoto
Copy link

@saiichihashimoto saiichihashimoto commented Oct 15, 2024

Why

JSON Schemas are an open and well used method of validating json configs and works well with existing tools. While the VSCode Expo plugin exists, working with existing tooling is ideal (and the expo already uses a json schema internally, so the additional lift is minimal).

How

Added to the Joi config

Closes #2600

Copy link

github-actions bot commented Oct 15, 2024

Subscribed to pull request

File Patterns Mentions
**/* @szdziedzic, @khamilowicz, @sjchmiela
packages/eas-json/** @szdziedzic, @khamilowicz, @sjchmiela

Generated by CodeMention

@saiichihashimoto
Copy link
Author

@szdziedzic do I need to reach out to a reviewer? What's the process for moving this forward?

@brentvatne
Copy link
Member

this seems reasonable to me, but i defer to @byCedric

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.

"$schema" is not allowed
2 participants