Skip to content

Conversation

@ssmadhavan006
Copy link

@ssmadhavan006 ssmadhavan006 commented Dec 2, 2025

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

Closes #485.

Description of Changes

commit - fb32869

  • Removed broken lint blocks from molecule/default/molecule.yml and molecule/local/molecule.yml
  • Edited run-qa-checks script by adding yamllint and ansible-lint
  • Add .yamllint.yml config to exclude virtual environments and and irrelevant files
  • Update CI 'ci.yml' with continue-on-error: true and added Added conditional test execution based on dependency installation success
  • Fixed indentation in molecule/resources/converge.yml
  • Removed trailing spaces in vars/debian-13.yml
  • Converted all YAML files from CRLF to LF line endings
  • Lint errors are no longer ignored, failures properly reported

commit 986f466

  • Updated Molecule callback configuration to replace deprecated yaml stdout_callback with
    ansible.builtin.default and result_format: yaml, resolving CI failures due to removal of
    community.general.yaml removal in v12+.

Screenshot

N/A

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug/qa] Lint errors are ignored

1 participant