Skip to content

Commit

Permalink
CI: Update and fix dev requirements (#702)
Browse files Browse the repository at this point in the history
  • Loading branch information
carlbuchmann authored Jul 19, 2024
1 parent 604a374 commit 1a761c9
Show file tree
Hide file tree
Showing 5 changed files with 31 additions and 8 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/pull-request-management.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ jobs:
- name: Set up Python 3
uses: actions/setup-python@v4
with:
python-version: '3.9'
python-version: '3.10'
- name: Install requirements
run: |
pip install -r ansible_collections/arista/cvp/requirements-dev.txt --upgrade
Expand Down Expand Up @@ -130,7 +130,7 @@ jobs:
strategy:
fail-fast: true
matrix:
python_version: ['3.9','3.10','3.11']
python_version: ['3.9','3.10','3.11','3.12']
steps:
- name: 'set environment variables'
run: |
Expand All @@ -142,8 +142,9 @@ jobs:
with:
python-version: ${{ matrix.python_version }}
- name: Install requirements
# Removing ansible-lint in requirement-dev-3.9 to allow requirements installation for Python 3.9.
run: |
pip install -r ansible_collections/arista/cvp/requirements-dev.txt
pip install -r ansible_collections/arista/cvp/requirements-dev-pytest.txt
pip install -r ansible_collections/arista/cvp/requirements.txt
- name: 'Execute pytest validation'
run: |
Expand Down Expand Up @@ -265,7 +266,7 @@ jobs:
strategy:
fail-fast: true
matrix:
python_version: ['3.9']
python_version: ['3.10']
steps:
- name: 'set environment variables'
run: |
Expand Down
2 changes: 1 addition & 1 deletion ansible_collections/arista/cvp/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ Please check the minimum version supported by your ansible installation on the [

**Ansible version:**

- ansible-core>=2.14.0,<2.17.0
- ansible-core>=2.15.0,<2.18.0

**3rd party Python libraries:**

Expand Down
2 changes: 1 addition & 1 deletion ansible_collections/arista/cvp/meta/runtime.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
requires_ansible: '>=2.14.0,<2.17.0'
requires_ansible: '>=2.15.0,<2.18.0'
plugin_routing:
modules:
cv_facts:
Expand Down
22 changes: 22 additions & 0 deletions ansible_collections/arista/cvp/requirements-dev-pytest.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# Temp requirements for pytest testing. Required ansible-lint version as it's not compatible with Python 3.9, so it is removed.
ansible-core>=2.15.0,<2.18.0
ansible-builder
galaxy-importer>=0.3.1
pycodestyle
flake8
pylint>=2.16.1
twine
pre-commit>=2.9.2
pre-commit-hooks>=3.3.0
identify>=1.4.20
docker
molecule>=3.2.0,<6.0
molecule-docker>=0.2.4
mitogen
pytest
pytest-cov>=2.11.1
pytest-html>=3.1.1
pytest-metadata>=1.11.0
pytest-dependency
jmespath
pytest-mock
4 changes: 2 additions & 2 deletions ansible_collections/arista/cvp/requirements-dev.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
ansible-core>=2.15.0,<2.17.0
ansible-core>=2.15.0,<2.18.0
ansible-builder
ansible-lint==6.22.2
ansible-lint>=24.7.0
galaxy-importer>=0.3.1
pycodestyle
flake8
Expand Down

0 comments on commit 1a761c9

Please sign in to comment.