Skip to content

Commit

Permalink
Migrate flake8 from workflow to lint script (apache#9062)
Browse files Browse the repository at this point in the history
Saw apache#9055 adds `flake8.sh` to the `docker/lint.sh` and remembered I started doing this in apache#8652, but never updated it after the Docker images were actually updated.
  • Loading branch information
Mousius authored and ylc committed Jan 13, 2022
1 parent a7f7c28 commit a9ccce0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,6 @@ jobs:
- uses: actions/checkout@v2
- name: Initialize submodules
run: git submodule update --recursive --init
- name: Lint Python
if: startsWith(matrix.os, 'macOS')
run: |
python3 -m pip install flake8
python3 -m flake8 . --count --select=E9,F63,F7 --show-source --statistics
- uses: actions/cache@v1
env:
CACHE_NUMBER: 0
Expand Down
4 changes: 2 additions & 2 deletions tests/lint/flake8.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@
# specific language governing permissions and limitations
# under the License.

# Disabled until docker images are rebuilt
# python3 -m flake8 . --count --select=E9,F63,F7 --show-source --statistics

python3 -m flake8 . --count --select=E9,F63,F7 --show-source --statistics

0 comments on commit a9ccce0

Please sign in to comment.