Skip to content

Commit

Permalink
refactor: Disable mypy and autoflake in CI and pre-commit hook
Browse files Browse the repository at this point in the history
- Commented out the mypy and autoflake checks in the CI workflow and pre-commit config files.
- The intention is to enable the mypy check again later.
  • Loading branch information
Pwuts committed Dec 2, 2023
1 parent c072c7d commit fe96f6d
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 17 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/autogpt-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,14 @@ jobs:
run: poetry run isort . --check
if: success() || failure()

- name: Check mypy formatting
run: poetry run mypy
if: success() || failure()

- name: Check for unused imports and pass statements
run: |
cmd="autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests"
poetry run $cmd --check || (echo "You have unused imports or pass statements, please run '${cmd} --in-place'" && exit 1)
# - name: Check mypy formatting
# run: poetry run mypy
# if: success() || failure()

# - name: Check for unused imports and pass statements
# run: |
# cmd="autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests"
# poetry run $cmd --check || (echo "You have unused imports or pass statements, please run '${cmd} --in-place'" && exit 1)

test:
permissions:
Expand Down
18 changes: 9 additions & 9 deletions autogpts/autogpt/.pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,18 +22,18 @@ repos:
- id: black
language_version: python3.10

- repo: https://github.com/pre-commit/mirrors-mypy
rev: 'v1.3.0'
hooks:
- id: mypy
# - repo: https://github.com/pre-commit/mirrors-mypy
# rev: 'v1.3.0'
# hooks:
# - id: mypy

- repo: local
hooks:
- id: autoflake
name: autoflake
entry: autoflake --in-place --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests
language: python
types: [ python ]
# - id: autoflake
# name: autoflake
# entry: autoflake --in-place --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests
# language: python
# types: [ python ]
- id: pytest-check
name: pytest-check
entry: pytest --cov=autogpt tests/unit
Expand Down

0 comments on commit fe96f6d

Please sign in to comment.