travis-ci / travis-yaml

parses, normalizes, validates and serializes your .travis.yml
http://yaml.travis-ci.org/
MIT License
170 stars 66 forks source link

multiple values are reported invalid but are ok on actual build #100

Open raphaelstolt opened 7 years ago

raphaelstolt commented 7 years ago

When validating this .travis.yml against Travis WebLint I get the following lint error message:

in matrix.include.env section: does not support multiple values, dropping LINT=true

Running it on Travis CI doesn't drop the LINT environment variable.

Linting should be in sync with the acual build behaviour.

Also the misleading lint error message might have people refactoring their Travis CI configuration in a readability decreasing way.

# fails validation, good readability 
matrix:
  include:
    - php: 7.0
      env:
      - DISABLE_XDEBUG=true
      - LINT=true

vs

# passes validation, decreased readability 
matrix:
  include:
    - php: 7.0
      env: DISABLE_XDEBUG=true LINT=true