Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --no-warnings flag #19 #20

Merged
merged 1 commit into from
May 26, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add --no-warnings flag #19
  • Loading branch information
ibiqlik committed May 26, 2021
commit dcecd0c0b0d970013c47993fd7931c681523f901
10 changes: 9 additions & 1 deletion .github/workflows/lint.yml
Original file line number Diff line number Diff line change
@@ -18,7 +18,15 @@ jobs:
trailing-spaces:
level: warning

- name: lint all (but pass)
- name: lint all - no warnings (continue on error)
continue-on-error: true
uses: ./
with:
file_or_dir: test
strict: true
no_warnings: true

- name: lint all (continue on error)
continue-on-error: true
uses: ./
with:
5 changes: 3 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
@@ -19,8 +19,9 @@ Simple as:
- `file1.yaml`
- `file1.yaml file2.yaml`
- `kustomize/**/*.yaml mychart/*values.yaml`
- `format` - Format for parsing output [parsable,standard,colored,github,auto] (default: parsable)
- `strict` - Return non-zero exit code on warnings as well as errors [true,false] (default: false)
- `format` - Format for parsing output `[parsable,standard,colored,github,auto] (default: parsable)`
- `strict` - Return non-zero exit code on warnings as well as errors `[true,false] (default: false)`
- `no_warnings` - Output only error level problems `[true,false] (default: false)`

**Note:** If `.yamllint` configuration file exists in your root folder, yamllint will automatically use it.

5 changes: 5 additions & 0 deletions action.yml
Original file line number Diff line number Diff line change
@@ -20,6 +20,10 @@ inputs:
description: 'Return non-zero exit code on warnings as well as errors'
required: false
default: "false"
no_warnings:
description: 'Output only error level problems'
required: false
default: "false"

runs:
using: 'composite'
@@ -32,6 +36,7 @@ runs:
INPUT_CONFIG_DATA: ${{ inputs.config_data }}
INPUT_FORMAT: ${{ inputs.format }}
INPUT_STRICT: ${{ inputs.strict }}
INPUT_NO_WARNINGS: ${{ inputs.no_warnings }}

branding:
color: blue
4 changes: 4 additions & 0 deletions entrypoint.sh
Original file line number Diff line number Diff line change
@@ -18,6 +18,10 @@ if [[ "$INPUT_STRICT" == "true" ]]; then
options+=(-s)
fi

if [[ "$INPUT_NO_WARNINGS" == "true" ]]; then
options+=(--no-warnings)
fi

# Enable globstar so ** globs recursively
shopt -s globstar
# Use the current directory by default