Skip to content

Commit

Permalink
Merge pull request #2 from fontivan/sskeard/yamllint
Browse files Browse the repository at this point in the history
Add yamllint, Fix lint for makefile.yml
  • Loading branch information
fontivan authored Apr 21, 2024
2 parents 791db80 + e534e6f commit 17d2d92
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 8 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/makefile.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
---
name: Makefile CI

on:
"on":
push:
branches: [ "main" ]
branches: ["main"]
pull_request:
branches: [ "main" ]
branches: ["main"]

jobs:
build:

runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3

- name: Run make all
run: make all
- uses: actions/checkout@v3
- name: Run make all
run: make all
6 changes: 5 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ PY = python3
VENV = venv
BIN=$(ROOT_DIR)/$(VENV)/bin

all: bashate shellcheck test
all: bashate shellcheck test yamllint

$(VENV): requirements.txt
$(PY) -m venv $(VENV)
Expand All @@ -29,6 +29,10 @@ shellcheck: $(VENV)
test:
$(ROOT_DIR)/test/test.sh

.PHONY: yamllint
yamllint: $(VENV)
$(BIN)/yamllint .

clean:
rm -rf $(VENV)
find . -type f -name *.pyc -delete
Expand Down
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
bashate
shellcheck-py
yamllint

0 comments on commit 17d2d92

Please sign in to comment.