roots / trellis-cli

A CLI to manage Trellis projects
https://roots.io/trellis/
MIT License
166 stars 25 forks source link

Fix use of reserved characters in wp-cli.yml #398

Closed kellymears closed 1 year ago

kellymears commented 1 year ago

I don't think the generated wp-cli.yml is compliant with the yml spec.

The “@” (x40, at) and “`” (x60, grave accent) are reserved for future use.

Reserved indicators can't start a plain scalar.

yaml spec (1.2.2)

yamllint and vscode's default YAML checks report errors, so I think I might be on to something.

test.yml
  1:1       error    syntax error: found character '@' that cannot start any token (syntax)
swalkinshaw commented 1 year ago

alias_test.go needs to be updated but otherwise this looks good 👍

chrillep commented 1 year ago

@swalkinshaw when will this be released? 😅

swalkinshaw commented 1 year ago

@chrillep I'll do one this week (maybe tonight).