Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable warnings related to the use of assert in the tests directory as reported with lint-vetting #792

Merged
merged 8 commits into from
Jan 25, 2022
Merged

Disable warnings related to the use of assert in the tests directory as reported with lint-vetting #792

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
ffac9be
assert
cidrblock Jan 23, 2022
b6c0757
Update .flake8
cidrblock Jan 23, 2022
e66ef99
Update .flake8
cidrblock Jan 23, 2022
1c921d9
Sort the entries in the flake8 `per_file_ignore` list (#794)
cidrblock Jan 24, 2022
3170caa
assert
cidrblock Jan 23, 2022
e8fe885
update
cidrblock Jan 24, 2022
6b5ae61
wut
cidrblock Jan 24, 2022
f227d66
explain why tests
cidrblock Jan 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion .flake8
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,18 @@ inline-quotes = "
max-line-length = 100

# Allow certain violations in certain files:
# Please keep this list sorted, as it will be easier for others to find and add entries in the future
# Please keep both sections of this list sorted, as it will be easier for others to find and add entries in the future
per-file-ignores =
# The following ignores have been researched and should be considered permanent
# each should be preceeded with an explanation of each of the error codes
# If other ignores are added for a specific file in the section following this,
# these will need to be added to that line as well.

# S101: Allow the use of assert within the tests directory, since tests require it.
tests/**.py: S101

# The following were present during the initial implementation.
# They are expected to be fixed and unignored over time.
docs/_ext/single_sourced_data.py: D400, D403, DAR101, DAR201, N817, P103, WPS110, WPS111, WPS201, WPS202, WPS210, WPS213, WPS221, WPS229, WPS231, WPS237, WPS305, WPS318, WPS323, WPS331, WPS336, WPS347, WPS407, WPS432, WPS440, WPS441, WPS453, WPS504
docs/_ext/spelling_stub_ext.py: DAR101, DAR201
docs/conf.py: WPS221, WPS226, WPS301, WPS305, WPS323, WPS420, WPS429, WPS433
webknjaz marked this conversation as resolved.
Show resolved Hide resolved
Expand Down