1
0
Fork 0
mirror of synced 2024-12-21 06:01:07 -05:00

feat(yamllint): include for this repo and apply rules throughout

* Semi-automated using `ssf-formula` (v0.5.0)
* Fix errors shown below:

```bash
sudoers-formula$ $(grep "\- yamllint" .travis.yml | sed -e "s:^\s\+-\s\(.*\):\1:")
pillar.example
  6:23      warning  truthy value should be one of [false, true]  (truthy)

test/salt/pillar/kitchen.sls
  3:1       warning  missing document start "---"  (document-start)
  7:1       error    too many blank lines (1 > 0)  (empty-lines)

test/salt/pillar/default.sls
  1:1       warning  missing document start "---"  (document-start)
  3:23      warning  truthy value should be one of [false, true]  (truthy)
```
This commit is contained in:
Imran Iqbal 2019-08-06 20:56:57 +01:00
parent 9bfca090ed
commit 8d08719844
No known key found for this signature in database
GPG key ID: 6D8629439D2B7819
6 changed files with 36 additions and 7 deletions

View file

@ -3,7 +3,7 @@
--- ---
stages: stages:
- test - test
- commitlint - lint
- name: release - name: release
if: branch = master AND type != pull_request if: branch = master AND type != pull_request
@ -45,16 +45,21 @@ script:
jobs: jobs:
include: include:
# Define the commitlint stage # Define the `lint` stage (runs `yamllint` and `commitlint`)
- stage: commitlint - stage: lint
language: node_js language: node_js
node_js: lts/* node_js: lts/*
before_install: skip before_install: skip
script: script:
# Install and run `yamllint`
- pip install --user yamllint
# yamllint disable-line rule:line-length
- yamllint -s . .yamllint pillar.example test/salt/pillar/kitchen.sls test/salt/pillar/default.sls
# 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
# Define the release stage that runs semantic-release # Define the release stage that runs `semantic-release`
- stage: release - stage: release
language: node_js language: node_js
node_js: lts/* node_js: lts/*

16
.yamllint Normal file
View file

@ -0,0 +1,16 @@
# -*- coding: utf-8 -*-
# vim: ft=yaml
---
# Extend the `default` configuration provided by `yamllint`
extends: default
# Files to ignore completely
# 1. All YAML files under directory `node_modules/`, introduced during the Travis run
ignore: |
node_modules/
rules:
line-length:
# Increase from default of `80`
# Based on https://github.com/PyCQA/flake8-bugbear#opinionated-warnings (`B950`)
max: 88

View file

@ -3,7 +3,7 @@
--- ---
sudoers: sudoers:
# By default the main sudoers file is managed by this formula (False to skip) # By default the main sudoers file is managed by this formula (False to skip)
manage_main_config: True manage_main_config: true
users: users:
johndoe: johndoe:
- 'ALL=(ALL) ALL' - 'ALL=(ALL) ALL'

View file

@ -1,3 +1,6 @@
# -*- coding: utf-8 -*-
# vim: ft=yaml
---
name: default name: default
title: sudoers formula title: sudoers formula
maintainer: SaltStack Formulas maintainer: SaltStack Formulas

View file

@ -1,6 +1,9 @@
# -*- coding: utf-8 -*-
# vim: ft=yaml
---
sudoers: sudoers:
# By default the main sudoers file is managed by this formula (False to skip) # By default the main sudoers file is managed by this formula (False to skip)
manage_main_config: True manage_main_config: true
users: users:
johndoe: johndoe:
- 'ALL=(ALL) ALL' - 'ALL=(ALL) ALL'

View file

@ -1,7 +1,9 @@
# -*- coding: utf-8 -*-
# vim: ft=yaml
---
# Pillar needed to run Inspec inside Docker image using sudo to authenticate # Pillar needed to run Inspec inside Docker image using sudo to authenticate
# Must be added to pillar of all test suites # Must be added to pillar of all test suites
sudoers: sudoers:
users: users:
kitchen: kitchen:
- 'ALL=(root) NOPASSWD: ALL' - 'ALL=(root) NOPASSWD: ALL'