Skip to content

Commit 35917a0

Browse files
authored
Merge pull request #2 from voidquark/add-lint
add ansible and yaml lint
2 parents dfcb573 + c737b74 commit 35917a0

File tree

7 files changed

+44
-3
lines changed

7 files changed

+44
-3
lines changed

.ansible-lint

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
skip_list:
2+
- 'yaml'
3+
- 'package-latest'

.github/workflows/lint.yaml

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
---
2+
name: Lint
3+
# yamllint disable-line rule:truthy
4+
on:
5+
push:
6+
branches:
7+
- main
8+
pull_request:
9+
branches:
10+
- main
11+
12+
jobs:
13+
lint:
14+
name: Linting
15+
runs-on: ubuntu-latest
16+
steps:
17+
- name: Checkout
18+
uses: actions/checkout@v4
19+
20+
- name: Install dependencies
21+
run: pip3 install yamllint
22+
23+
- name: Run Yaml Lint
24+
run: |
25+
yamllint .
26+
27+
- name: Run Ansible Lint
28+
uses: ansible/ansible-lint@main

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
.vscode

.yamllint.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
extends: default
3+
4+
rules:
5+
line-length:
6+
max: 230
7+
level: warning

defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ el_patching_auto_reboot: false
99
# el_patching_exclude_packages:
1010
# - tar
1111
# - zip
12-
# el_patching_update_cache: true
12+
# el_patching_update_cache: true

meta/main.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
---
12
galaxy_info:
23
author: voidquark
34
description: Apply Linux OS patches
@@ -14,4 +15,4 @@ galaxy_info:
1415
- name: EL
1516
versions:
1617
- "8"
17-
- "9"
18+
- "9"

tasks/main.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
cmd: needs-restarting --reboothint
3838
register: __el_patching_need_restart
3939
failed_when: __el_patching_need_restart.rc > 1
40+
changed_when: false
4041

4142
- name: Inform user if reboot is required
4243
ansible.builtin.debug:
@@ -46,4 +47,4 @@
4647
- name: Reboot host
4748
ansible.builtin.reboot:
4849
reboot_timeout: "{{ el_patching_reboot_timeout }}"
49-
when: el_patching_auto_reboot | bool and __el_patching_need_restart.rc == 1
50+
when: el_patching_auto_reboot | bool and __el_patching_need_restart.rc == 1

0 commit comments

Comments
 (0)