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

`dist:` in `matrix` section marked as "dropped" by linter, works 👌 in build #118

Open ntwb opened 7 years ago

ntwb commented 7 years ago

The following fails at https://lint.travis-ci.org/ and returns an error: in matrix.include section: unexpected key dist, dropping. The build passes though and the PHP 5.2 & 5.3 jobs do use Precise as the distro in the build.

sudo: false
language: php
dist: trusty
php:
  - 7.1
  - 7.0
  - 5.6
  - 5.5
  - 5.4
env:
  - WP_VERSION=master
  - WP_VERSION=4.7

# Build matrix options
matrix:
  include:
    - php: 5.3
      env: WP_VERSION=4.7
      dist: precise
    - php: 5.2
      env: WP_VERSION=master
      dist: precise
ntwb commented 7 years ago

FYI: It appears to throw a similar error for dist, sudo, group, and addons:

in matrix.include section: unexpected key dist, dropping in matrix.include section: unexpected key sudo, dropping in matrix.include section: unexpected key group, dropping in matrix.include section: unexpected key addons, dropping

gdamore commented 7 years ago

This is quite annoying -- the recommended "linter" for YAML files actually fails on examples from the Travis documentation.