Skip to content

Commit

Permalink
Upgrade GitHub Actions and make bandit, codespell, and pytest mandato…
Browse files Browse the repository at this point in the history
…ry (#835)

* Upgrade GitHub Actions

* Fix typo discovered by codespell

* Update lint_python.yml
  • Loading branch information
cclauss committed Oct 6, 2022
1 parent a70e13f commit c3dda12
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 9 deletions.
18 changes: 10 additions & 8 deletions .github/workflows/lint_python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,22 +4,24 @@ jobs:
lint_python:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v2
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
python-version: 3.x
- run: pip install --upgrade pip wheel
- run: pip install bandit black codespell flake8 flake8-2020 flake8-bugbear
flake8-comprehensions isort mypy pytest pyupgrade safety
- run: bandit --recursive --skip B101 . || true # B101 is assert statements
- run: bandit --recursive --skip B101,B105,B106,B107,B324 .
- run: black --check . || true
- run: codespell || true # --ignore-words-list="" --skip="*.css,*.js,*.lock"
- run: codespell # --ignore-words-list="" --skip="*.css,*.js,*.lock"
- run: flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
- run: flake8 . --count --exit-zero --max-complexity=10 --max-line-length=88
--show-source --statistics
- run: isort --check-only --profile black . || true
- run: pip install -r requirements.txt || pip install --editable . || true
- run: pip install -r requirements-test.txt
- run: pip install --editable .
- run: mkdir --parents --verbose .mypy_cache
- run: mypy --ignore-missing-imports --install-types --non-interactive . || true
- run: pytest . || true
- run: pytest --doctest-modules . || true
- run: shopt -s globstar && pyupgrade --py36-plus **/*.py || true
- run: pytest
- run: shopt -s globstar && pyupgrade --py37-plus **/*.py || true
- run: safety check
2 changes: 1 addition & 1 deletion oauthlib/openid/connect/core/request_validator.py
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ def finalize_id_token(self, id_token, token, token_handler, request):
Token MUST NOT be accepted by the RP when performing
authentication with the OP.
Additionals claims must be added, note that `request.scope`
Additional claims must be added, note that `request.scope`
should be used to determine the list of claims.
More information can be found at `OpenID Connect Core#Claims`_
Expand Down

0 comments on commit c3dda12

Please sign in to comment.