mirror of
https://github.com/benbusby/whoogle-search.git
synced 2026-04-25 20:25:51 +03:00
[PR #161] [MERGED] PEP-8 enforcement #805
Labels
No labels
Fixed (Pending PR Merge)
Stale
bug
enhancement
enhancement
good first issue
help wanted
keep-open
needs more info
pull-request
question
theme
unfortunate
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/whoogle-search#805
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/benbusby/whoogle-search/pull/161
Author: @benbusby
Created: 12/17/2020
Status: ✅ Merged
Merged: 12/17/2020
Merged by: @benbusby
Base:
develop← Head:feature/pep8📝 Commits (8)
06d1274PEP-8: Fix formatting for request class and routes3350e54PEP-8: Update max line len to 79cb0b886PEP-8: Fix formatting in config model and utilsdd56ca9PEP-8: pep8 -> pycodestyle, fix init92d94d9PEP-8: Fix filter class formatting2533b7fPEP-8: Fix test formatting9db8ff5PEP-8: Add github action for pep83047011Merge branch 'develop' into feature/pep8📊 Changes
10 files changed (+330 additions, -158 deletions)
View changed files
➕
.github/workflows/pep8.yml(+22 -0)📝
app/__init__.py(+24 -9)📝
app/filter.py(+47 -26)📝
app/models/config.py(+14 -10)📝
app/request.py(+51 -28)📝
app/routes.py(+120 -55)📝
app/utils/filter_utils.py(+10 -7)📝
app/utils/routing_utils.py(+34 -18)📝
requirements.txt(+1 -0)📝
test/test_results.py(+7 -5)📄 Description
The formatting of the code base has been a bit inconsistent. This wraps up all remaining PEP-8 formatting issues and introduces a new CI workflow for ensuring all pushes adhere to the proper format.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.