action #162044
Updated by livdywan 4 months ago
## Motivation See https://gitlab.suse.de/openqa/salt-pillars-openqa/-/merge_requests/832?commit_id=8412fb44dee81426a40e1f114aed3e62ca7e0667#note_633754 https://gitlab.suse.de/openqa/salt-pillars-openqa/-/merge_requests /832?commit_id=8412fb44dee81426a40e1f114aed3e62ca7e0667#note_633754 With the merge request we align both repos to use the same YAML style rules but it would be better to explicitly check for indendation. So we should enable indendation check, at best by removing any customization in the yamllint config file, in both salt-states-openqa+salt-pillars-openqa and apply the according necessary style adaptations in both repos. ## Acceptance criteria * **AC1:** Line https://gitlab.suse.de/openqa/salt-states-openqa/-/blob/master/.yamllint?ref_type=heads#L6 is absent * **AC2:** yamllint ensures proper indendation in salt-states-openqa * **AC3:** Same as AC1+AC2 AC1+AC3 but for salt-pillars-openqa ## Suggestions * Remove https://gitlab.suse.de/openqa/salt-states-openqa/-/blob/master/.yamllint?ref_type=heads#L6, call yamllint, see where it fails and adapt the code * As necessary add a custom rule but at best we can just live with the default * Apply the same for salt-pillars-openqa