Fix broken Yamllint Github Action

nextcloud / ansible-collection-nextcloud-admin

The ansible galaxy for your nextcloud administrative needs.

https://galaxy.ansible.com/nextcloud/admin

BSD 2-Clause "Simplified" License

138 stars 77 forks source link

Fix broken Yamllint Github Action #286

Closed staticdev closed 1 year ago

staticdev commented 1 year ago

Closes #284

This PR introduces following changes:

Replaces broken yamlint GA with yamllint installation and execution steps
Merges both yamllint and ansible-lint workflows for easier maintenance of linting

staticdev commented 1 year ago

@aalaesar @wiktor2200 I would like to merge this PR since lint part is working. Other errors are because of NC27 installation, it seems that now OCC changes/removed some argument we use on php occ maintenance:install.

Then I will follow up the work on https://github.com/nextcloud/ansible-collection-nextcloud-admin/pull/285

wiktor2200 commented 1 year ago

Hi @staticdev. Sorry for quite a long time, no response from my side. Thanks a lot for keeping it up-to-date and actively maintained, I appreciate that :) I've left small comment in code. But approving anyways if you agree with my comment just apply suggestion if not feel free to merge it as it is :)

staticdev commented 1 year ago

I've left small comment, but I think it's optional and can be changed later. Up-to you @staticdev :)

Thanks @wiktor2200, I added your change, now I need approval again XD