From ae19c9723ba2007ea28dd57c4df65eeea41be3ae Mon Sep 17 00:00:00 2001 From: Eric Cousineau Date: Fri, 12 Feb 2021 12:25:08 -0500 Subject: [PATCH] tests: Remove usage of --flake8 flag (#5909) * tests: Remove usage of --flake8 flag * Remove commented line Co-authored-by: Carlos Mocholi --- .github/CONTRIBUTING.md | 2 +- Makefile | 2 +- requirements/test.txt | 1 - 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 98321b2095..4eed846b28 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -202,7 +202,7 @@ This is useful if you do not test against all required dependency versions. **Docker:** Another option is utilize the [pytorch lightning cuda base docker image](https://hub.docker.com/repository/docker/pytorchlightning/pytorch_lightning/tags?page=1&name=cuda). You can then run: ```bash -python -m pytest pytorch_lightning tests pl_examples -v --flake8 +python -m pytest pytorch_lightning tests pl_examples -v ``` ### Pull Request diff --git a/Makefile b/Makefile index a659d4a4b0..964b9ab103 100644 --- a/Makefile +++ b/Makefile @@ -23,7 +23,7 @@ test: clean # install APEX, see https://github.com/NVIDIA/apex#linux # use this to run tests - python -m coverage run --source pytorch_lightning -m pytest pytorch_lightning tests pl_examples -v --flake8 + python -m coverage run --source pytorch_lightning -m pytest pytorch_lightning tests pl_examples -v python -m coverage report docs: clean diff --git a/requirements/test.txt b/requirements/test.txt index 1ca085ef2a..2d47143ca5 100644 --- a/requirements/test.txt +++ b/requirements/test.txt @@ -2,7 +2,6 @@ coverage>=5.0 codecov>=2.1 pytest>=5.0 # pytest-cov -# pytest-flake8 flake8>=3.6 check-manifest twine==3.2