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

Validate YAML files #38

Closed
ruflin opened this issue May 4, 2020 · 5 comments · Fixed by #41
Closed

Validate YAML files #38

ruflin opened this issue May 4, 2020 · 5 comments · Fixed by #41
Assignees

Comments

@ruflin
Copy link
Contributor

ruflin commented May 4, 2020

All available yaml files in the packages should be validated for valid yaml to make sure packages only contain valid yaml.

An example where this would have been useful can be found here: elastic/package-registry#421

@ruflin
Copy link
Contributor Author

ruflin commented May 4, 2020

If we validate all yaml files, we should also validate all json files.

@ph
Copy link

ph commented Aug 25, 2020

@ycombinator I think its part of the validation that you are currently enforcing?

@ycombinator
Copy link
Contributor

Yes, this will ultimately be handled by the validation provided by the package-spec.

@ph
Copy link

ph commented Aug 25, 2020

Going to move the issue in that repository.

@ph ph transferred this issue from elastic/package-registry Aug 25, 2020
@mtojek mtojek assigned mtojek and unassigned ycombinator Aug 27, 2020
@mtojek
Copy link
Contributor

mtojek commented Aug 27, 2020

Let me try to handle this.

@mtojek mtojek closed this as completed in #41 Sep 1, 2020
rw-access pushed a commit to rw-access/package-spec that referenced this issue Mar 23, 2021
I assume this binary should never be checked in.
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 a pull request may close this issue.

4 participants