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.

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

fix linting #14

Merged
merged 5 commits into from
Oct 3, 2024
Merged

fix linting #14

merged 5 commits into from
Oct 3, 2024

Conversation

hitchhooker
Copy link
Contributor

    steps:
      - uses: actions/checkout@v3
      - name: Set up Python ${{ matrix.python-version }}
        uses: actions/setup-python@v4
        with:
          python-version: ${{ matrix.python-version }}
          cache: 'pip'
          cache-dependency-path: 'requirements*.txt'
      - name: Install dependencies and run linters
        run: |
          export PIP_NO_CLEAN=TRUE
          export PIP_PREFER_BINARY=TRUE
          python -m pip install --upgrade pip setuptools wheel
          if [ -f requirements-dev.txt ]; then
            python -m pip install -r requirements-dev.txt
          elif [ -f requirements.txt ]; then
            python -m pip install -r requirements.txt
          fi
          if [ -f requirements.yaml ]; then ansible-galaxy install -r requirements.yaml -p ./ext_roles; fi

          # Set up ansible vault
          echo $ANSIBLE_VAULT_PASSWORD > .vault_pass
          chmod 600 .vault_pass
          export ANSIBLE_VAULT_PASSWORD_FILE=.vault_pass

          # Run all commands within "ansible" directory
          inv syntaxcheck
          inv ansiblelint
          inv yamllint
          inv flake8
          inv black
          inv ansiblelater
        env:
          ANSIBLE_VAULT_PASSWORD: ${{ secrets.VAULT_PASS }}

we need to be bable to pass this test

@hitchhooker hitchhooker merged commit 88465f8 into master Oct 3, 2024
2 checks passed
@hitchhooker hitchhooker deleted the linting branch October 3, 2024 05:39
@hitchhooker hitchhooker restored the linting branch October 3, 2024 05:40
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.

1 participant