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.

Sign up for GitHub

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

Jump to bottom

Update ansible-lint action and revise configuration to scan all Ansible code #732

Merged
merged 17 commits into from
Dec 7, 2023
Merged

Update ansible-lint action and revise configuration to scan all Ansible code #732

merged 17 commits into from
Dec 7, 2023

Conversation

schurzi
Copy link
Contributor

@schurzi schurzi commented Dec 6, 2023

No description provided.

@schurzi
use new ansible-lint action
9862676 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
remove excludes
099145c 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi schurzi linked an issue Dec 6, 2023 that may be closed by this pull request
Extend ansible-lint testing to cover our test cases #731
Closed
@schurzi
extend action triggers
db437c7 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
exclude waiver files
2bbfe2d 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
extend action triggers
f450045 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
adjust exclude paths
6ce8b68 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@rndmh3ro rndmh3ro changed the title Update ansible-int action and revise configuration to scan all Ansible code Update ansible-lint action and revise configuration to scan all Ansible code Dec 6, 2023
@schurzi
fix lint findings
e00716d 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
fix paste error
5381a23 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
use ansible-lint to autofix problems
7e33ea0 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
manually fix remaining problems
422fb94 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
correct typo
dff224f 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
remove mysql config directory creation
bc8b5f6 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
re-add mysql config directory because it is needed by some os
62340c7 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
hold grub updates in vm
4bc5725 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
add prepare for archlinux
bf51b96 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
add prepare for archlinux
5843b03 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi
add noconfirm
1f0d2d5 
Signed-off-by: Martin Schurz <Martin.Schurz@telekom.de>
@schurzi schurzi marked this pull request as ready for review December 7, 2023 06:52
@schurzi schurzi requested a review from rndmh3ro December 7, 2023 06:52
@schurzi schurzi merged commit e37f01d into master Dec 7, 2023
92 of 93 checks passed
@schurzi schurzi deleted the lintall branch December 7, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rndmh3ro rndmh3ro rndmh3ro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Extend ansible-lint testing to cover our test cases
2 participants
@schurzi @rndmh3ro