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

add workflow to validate yaml files #402

Closed
wants to merge 1 commit into from

Conversation

andibraeu
Copy link
Contributor

we maybe want to prevent corrupt html results to be published.

to make use of this, I guess branch protection has to be enabled and a required check validate-yaml should be configured

@coderbyheart
Copy link
Collaborator

I like the idea of validating the YAML files, however as you can see from your check there are a lot of false positives that do not actually cause an error.

I suggest to implement validation based on some content rules (for example using a JSON schema with the YAML files converted to JSON).

@andibraeu
Copy link
Contributor Author

andibraeu commented Nov 15, 2023

ok, that's why I set the level to warning for newline and trailing spaces. It is also possible to disable all these warnings

But let me try https://github.com/marketplace/actions/json-yaml-validate first

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.

2 participants