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.

Sign up for GitHub

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

Jump to bottom

Add YAML validation #857

Closed
TravisCarden opened this issue Dec 22, 2016 · 3 comments
Closed

Add YAML validation #857

TravisCarden opened this issue Dec 22, 2016 · 3 comments
Assignees
@grasmash
Labels
Enhancement A feature or feature request

Comments

@TravisCarden
Copy link
Contributor

Since syntax errors in YAML syntax errors can be just as serious as in PHP, we should have a validation command for them (e.g., validate:yaml) and include it in validate:all.
@grasmash grasmash added the Enhancement A feature or feature request label Dec 27, 2016
@grasmash
Copy link
Contributor

Symfony's Yaml component provides a validation command that is available in 3.x. However, we are blocked on upgrading in #646 by hechoendrupal/drupal-console#2905

@jmolivas
Copy link
Contributor

@grasmash I am currently working on fixing this DrupalConsole issue.

@grasmash
Copy link
Contributor

@jmolivas Thanks so much for your help!

@grasmash grasmash removed the blocked label Jan 4, 2017
grasmash added a commit to grasmash/bolt that referenced this issue Jan 4, 2017
@grasmash
Fixes acquia#857: Adding YAML linting target.
48b517e
@grasmash grasmash self-assigned this Jan 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grasmash grasmash

Labels
Enhancement A feature or feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmolivas @grasmash @TravisCarden