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

feat: support more stencil.yaml paths, generate JSON schema #84

Merged

Conversation

jaredallard
Copy link
Member

We now generate JSON schema files from the Go types directly for
manifest.yaml and stencil.yaml. This allows us to be able to start
validating the configuration for stencil in editors and provide a
better user experience there. Eventually, this can cascade into other
schemas being supported.

I also added support for .stencil.yaml so we can match other
configuration file formats and keep it hidden (which we opt to do here
now).

We now generate JSON schema files from the Go types directly for
`manifest.yaml` and `stencil.yaml`. This allows us to be able to start
validating the configuration for `stencil` in editors and provide a
better user experience there. Eventually, this can cascade into other
schemas being supported.

I also added support for `.stencil.yaml` so we can match other
configuration file formats and keep it hidden (which we opt to do here
now).
Copy link

cloudflare-pages bot commented Jun 6, 2024

Deploying stencil with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1146243
Status: ✅  Deploy successful!
Preview URL: https://674e2ffb.stencil-1wj.pages.dev
Branch Preview URL: https://jaredallard-feat-support-mor.stencil-1wj.pages.dev

View logs

@jaredallard jaredallard requested a review from deregtd June 6, 2024 00:55
Copy link

codecov bot commented Jun 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Copy link
Collaborator

@deregtd deregtd left a comment

Choose a reason for hiding this comment

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

general thing is approved, but as i mentioned offline, i don't agree with having a hidden .stencil.yaml file. it's not a hill i'll die on, i just won't use it for any projects here or at my company.

@jaredallard jaredallard enabled auto-merge (squash) June 6, 2024 15:32
@jaredallard jaredallard merged commit a93018f into main Jun 6, 2024
3 checks passed
@jaredallard jaredallard deleted the jaredallard/feat/support-more-stencil-config-schema branch June 6, 2024 15:34
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.

None yet

2 participants