search

fluid-project / fluid-lint-all

Consolidated linting logic free from any particular build technology
BSD 3-Clause "New" or "Revised" License
0 stars 5 forks source link

Add YAML validation checks. #48

Closed greatislander closed 2 years ago

greatislander commented 2 years ago

Is your feature request related to a problem?

This would be handy, especially for linting GitHub Actions workflows.

Describe the solution you'd like

Not sure what the best JavaScript-based linter for YAML is but we should explore some options. A cursory search reveals:

Describe alternative solutions you've considered

Not applicable.

Additional context or notes

See https://github.com/fluid-project/infusion-docs/pull/212#issuecomment-1041464322.

the-t-in-rtf commented 2 years ago

@greatislander, is the solution you use in your IDE javascript based (or part of a "family" that has javascript bindings available)?

the-t-in-rtf commented 2 years ago

I looked at the two suggestions you had and a few others that were as stale as yaml-lint or worse. Although yaml-validator requires node@14, looks like the js-yaml dependency that does the heavy lifting works with node@12. Depending on the error structure, js-yaml may be adequate for what we need.