Browse Source

ci(travis): merge `rubocop` linter into main `lint` job

* Semi-automated using https://github.com/myii/ssf-formula/pull/65
tags/v1.2.1
Imran Iqbal 5 years ago
parent
commit
ec20e4cc7e
1 changed files with 5 additions and 16 deletions
  1. +5
    -16
      .travis.yml

+ 5
- 16
.travis.yml View File

- name: release - name: release
if: branch = master AND type != pull_request if: branch = master AND type != pull_request
jobs: jobs:
allow_failures:
- env: Lint_rubocop
fast_finish: true
include: include:
## Define the test stage that runs the linters (and testing matrix, if applicable) ## Define the test stage that runs the linters (and testing matrix, if applicable)


# Run all of the linters in a single job (except `rubocop`)
# Run all of the linters in a single job
- language: node_js - language: node_js
node_js: lts/* node_js: lts/*
env: Lint env: Lint
name: 'Lint: salt-lint, yamllint & commitlint'
name: 'Lint: salt-lint, yamllint, rubocop & commitlint'
before_install: skip before_install: skip
script: script:
# Install and run `salt-lint` # Install and run `salt-lint`
# Need at least `v1.17.0` for the `yaml-files` setting # Need at least `v1.17.0` for the `yaml-files` setting
- pip install --user yamllint>=1.17.0 - pip install --user yamllint>=1.17.0
- yamllint -s . - yamllint -s .
# Install and run `rubocop`
- gem install rubocop
- rubocop -d
# Install and run `commitlint` # Install and run `commitlint`
- npm install @commitlint/config-conventional -D - npm install @commitlint/config-conventional -D
- npm install @commitlint/travis-cli -D - npm install @commitlint/travis-cli -D
- commitlint-travis - commitlint-travis
# Run the `rubocop` linter in a separate job that is allowed to fail
# Once these lint errors are fixed, this can be merged into a single job
- language: node_js
node_js: lts/*
env: Lint_rubocop
name: 'Lint: rubocop'
before_install: skip
script:
# Install and run `rubocop`
- gem install rubocop
- rubocop -d


## Define the rest of the matrix based on Kitchen testing ## Define the rest of the matrix based on Kitchen testing
# Make sure the instances listed below match up with # Make sure the instances listed below match up with

Loading…
Cancel
Save