Skip to content

Commit

Permalink
Fixed small issue with python setup and ruff issue.
Browse files Browse the repository at this point in the history
  • Loading branch information
VatsalJagani committed Mar 15, 2024
1 parent feaf1af commit 36ec04e
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions .github/workflows/py_unit_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,9 @@ jobs:
- uses: actions/checkout@v4

- name: Set up Python
uses: actions/setup-python@v4
uses: actions/setup-python@v5
with:
python-version: '3.11'

- name: Display Python version
run: python -c "import sys; print(sys.version)"
Expand All @@ -34,6 +36,10 @@ jobs:
python -m pip install --upgrade pip
pip install -r tests/requirements.txt
- name: Linting with Ruff
run: ruff check --ignore=E721,E722,E401 --output-format=github --exclude=tests/test_app_repos --exclude=repodir/tests/test_app_repos .
# continue-on-error: true # to ignore ruff errors

- name: Run Python Unit Tests
env:
SPLUNKBASE_USERNAME_FOR_TEST: ${{ secrets.SPLUNKBASE_USERNAME }}
Expand All @@ -46,7 +52,3 @@ jobs:
pytest_results_file: "junit/test-results.xml"
pytest_cov_file: "coverage.xml"
if: ${{ always() }}

- name: Linting with Ruff
run: ruff check --ignore=E721,E722,E401 --output-format=github --exclude=tests/test_app_repos .
# continue-on-error: true # need to remove this later on to catch the errors anf fail the workflow on linting errors

0 comments on commit 36ec04e

Please sign in to comment.