Skip to content

Commit

Permalink
Merge pull request #18 from igormcsouza/hotfix
Browse files Browse the repository at this point in the history
Update CI to run lint also
  • Loading branch information
igormcsouza authored Mar 23, 2023
2 parents dfed8de + 2c9fb93 commit 3ec9c0f
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 5 deletions.
12 changes: 12 additions & 0 deletions .flake8
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
[flake8]
extend-ignore = E203, W391
exclude =
.git,
__pycache__,
docs/source/conf.py,
old,
build,
dist,
.venv
max-complexity = 10

13 changes: 8 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,11 @@ jobs:
web:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- name: Build the docker image
run: docker-compose build
- name: Use pytest to execute the tests
run: docker-compose run web pytest
- uses: actions/checkout@v3
- name: Install Requirements
run: pip install -r requirements.txt
- name: Run Linter on python code with flake8
run: flake8
- name: Use pytest to run unittests
run: pytest

2 changes: 2 additions & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
fastapi==0.68.1
uvicorn==0.15.0
GoogleNews==1.6.6

# Test packages
pytest==6.2.5
flake8==6.0.0
requests

0 comments on commit 3ec9c0f

Please sign in to comment.