String "yes" is interpreted wrong

[remcohaszing]

Given a Kubernetes MySQL deployment file:

apiVersion: extensions/v1beta1
kind: Deployment
# ...
spec:
  # ...
  template:
    # ...
    spec:
      containers:
        - env:
            - name: MYSQL_RANDOM_ROOT_PASSWORD
              value: "yes"
          # ...

The value yes is a string. However, I am getting the following error in VSCode:

Incorrect type. Expected "string".

Probably the string "yes" is incorrectly marked as a boolean.

[JPinkney]

It is marked as boolean because yaml spec says "yes" should be marked as boolean http://yaml.org/type/bool.html

[remcohaszing]

It is marked as boolean because yaml spec says "yes" should be marked as boolean http://yaml.org/type/bool.html

No, it doesn’t. In the definition where yes is quoted, the quote is used as punctuation. The regular expression doesn’t contain any quotes. Also libraries and tools seem to agree that "yes" should be treated as a string.

> require('js-yaml').load('"yes"')
'yes'

>>> import yaml
>>> yaml.load('"yes"')
'yes'

The example in the original issue is part of a valid Kubernetes configuration file. I need to define an environment variable MYSQL_RANDOM_ROOT_PASSWORD=yes. This is how it is done. All tools accept it, except vscode-yaml.

[JPinkney]

It will be fixed in tomorrows release.

[tristan957]

same as "no". Not sure if that was fixed as well

[tristan957]

Looks like it was. thanks :)

[tomasaschan]

With the latest released version of this extension I'm still seeing this error (for the mapping value: 'Yes'). Was this fix released yet? If not, when can I expect an update? :)

[JPinkney]

The fix hasn't been released yet. I won't be able to do a release until December 20th but that release will include the fix.

[JPinkney]

This should be fixed now!