Remove pep8 workflow from ci
PEP-8 enforcements in the project are more of an annoyance than anything. It doesn't really seem to add much value, and adds a lot of friction to pull requests from developers who aren't familiar with the style guide. Stylistic enforcements should just be done during PRs if necessary (or a different style guide should be enforced).
This commit is contained in:
parent
73dd5b80b5
commit
8e867a5ace
21
.github/workflows/pep8.yml
vendored
21
.github/workflows/pep8.yml
vendored
|
@ -1,21 +0,0 @@
|
||||||
name: pep8
|
|
||||||
|
|
||||||
on: [push, pull_request]
|
|
||||||
|
|
||||||
jobs:
|
|
||||||
test:
|
|
||||||
runs-on: ubuntu-latest
|
|
||||||
steps:
|
|
||||||
- uses: actions/checkout@v2
|
|
||||||
- name: Set up Python
|
|
||||||
uses: actions/setup-python@v2
|
|
||||||
with:
|
|
||||||
python-version: '3.x'
|
|
||||||
- name: Install dependencies
|
|
||||||
run: |
|
|
||||||
python -m pip install --upgrade pip
|
|
||||||
pip install pycodestyle
|
|
||||||
- name: Run pycodestyle
|
|
||||||
run: |
|
|
||||||
pycodestyle --show-source --show-pep8 app/*
|
|
||||||
pycodestyle --show-source --show-pep8 test/*
|
|
|
@ -4,7 +4,6 @@
|
||||||
[](https://opensource.org/licenses/MIT)
|
[](https://opensource.org/licenses/MIT)
|
||||||
[](https://github.com/benbusby/whoogle-search/actions/workflows/tests.yml)
|
[](https://github.com/benbusby/whoogle-search/actions/workflows/tests.yml)
|
||||||
[](https://github.com/benbusby/whoogle-search/actions/workflows/buildx.yml)
|
[](https://github.com/benbusby/whoogle-search/actions/workflows/buildx.yml)
|
||||||
[](https://github.com/benbusby/whoogle-search/actions?query=workflow%3Apep8)
|
|
||||||
[](https://codebeat.co/projects/github-com-benbusby-shoogle-master)
|
[](https://codebeat.co/projects/github-com-benbusby-shoogle-master)
|
||||||
[](https://hub.docker.com/r/benbusby/whoogle-search)
|
[](https://hub.docker.com/r/benbusby/whoogle-search)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user