Skip to content

Commit

Permalink
Added yamllint, fixed current linting errors, added github action
Browse files Browse the repository at this point in the history
  • Loading branch information
priyanshikhetwani committed Nov 29, 2024
1 parent 089ca79 commit ecc3b86
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 15 deletions.
9 changes: 1 addition & 8 deletions .github/workflows/validate-yaml-lint.yaml
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
name: YamlLint
on:
push:
branches:
- main
pull_request:
branches:
- main
workflow_dispatch:
on: [push, pull_request]
jobs:
yamllint:
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion .yamllint
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ rules:

# Rules to control lines
line-length:
max: 150 # Maximum 150 characters per line
max: 200 # Maximum 150 characters per line
allow-non-breakable-words: true # Allow long words without breaks
allow-non-breakable-inline-mappings: true # Allow long inline mappings without breaks
empty-lines:
Expand Down
6 changes: 0 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -551,12 +551,6 @@ endif
yamllint:
@docker run --rm $$(tty -s && echo "-it" || echo) -v $(PWD):/data cytopia/yamllint:latest /data --config-file /data/.yamllint --no-warnings

.PHONY: lint-yaml-no-warnings
lint-yaml-no-warnings: ## Run yamllint and won’t output warning level problems
@echo "Running yamllint and won't output warning level problems..."
@yamllint . --no-warnings
@echo "Linting Yaml files completed"

## --------------------------------------
## Cleanup / Verification
## --------------------------------------
Expand Down

0 comments on commit ecc3b86

Please sign in to comment.