Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-12-01 08:42:13 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity
ccd11e95a6
gecko-dev/.flake8

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

127 lines
4.5 KiB
Plaintext
Raw Normal View History

Bug 1270506 - [mozlint] Add python flake8 linter, r=smacleod For now, only the following two directories will be linted: python/mozlint tools/lint New directories can be added by adding them to the 'include' directive in tools/lint/flake8.lint. They all default to the configuration specified in topsrcdir/.flake8. Subdirectories can override this configuration by creating their own .flake8 file. MozReview-Commit-ID: Eag48Lnkp3l --HG-- extra : rebase_source : 6d98c9fef80055a48cc2622848aa04c3045b747e
2016-05-05 21:21:12 +00:00
[flake8]
max-line-length = 99
Bug 1395126 - Support cascading configuration for flake8, r=bc This allows .flake8 files to override one another, and fixes a pretty bad known bug with our flake8 implementation. For example, say we have a .flake8 file at: /foo/.flake8 Before this patch, if we ran |mach lint foo/bar|, the configuration defined in that .flake8 file wouldn't get picked up. It would only work if running the specific directory that contains it, e.g |mach lint foo|. This change additionally allows multiple .flake8 files to be used. So if there's one defined at both: /.flake8 /foo/.flake8 Then running |mach lint foo/bar| will first apply the root .flake8, then the one under /foo (overriding earlier configuration). This bug still doesn't make flake8 configuration perfect though. Any directory containing a .flake8 file still needs to be explicitly listed in the "include" section of /tools/lint/flake8.yml. Otherwise in the example above, if running |mach lint /|, it wouldn't be able to find /foo/.flake8. This is a hard problem and is likely best solved by fixing flake8's upstream configuration handling. Unfortunately this means we still can't switch from a whitelist to a blacklist. MozReview-Commit-ID: 3DZAi1QHYYo --HG-- extra : rebase_source : 51298c5847f6c2792581d9b312c87b70fa716ee1
2017-08-29 21:32:31 +00:00
exclude =
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
# These paths should be triaged and either fixed or moved to the list below.
devtools/shared,
Backed out changeset 02b8e04c764c (bug 1622677) for causing multiple failures related to WebIDL. CLOSED TREE
2022-12-17 10:00:00 +00:00
dom/bindings,
Bug 1567642 - [lint.flake8] Skip remaining flake8 under Python 3 lint issues r=gbrown Depends on D45417 Differential Revision: https://phabricator.services.mozilla.com/D46352 --HG-- extra : moz-landing-system : lando
2019-09-24 14:46:38 +00:00
ipc/ipdl/ipdl,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
layout/base/tests/marionette,
layout/reftests/border-image,
layout/reftests/fonts,
layout/reftests/w3c-css,
layout/style,
media/libdav1d/generate_source.py,
moz.configure,
netwerk/dns/prepare_tlds.py,
netwerk/protocol/http/make_incoming_tables.py,
python/l10n/fluent_migrations,
Bug 1567642 - [lint.flake8] Skip remaining flake8 under Python 3 lint issues r=gbrown Depends on D45417 Differential Revision: https://phabricator.services.mozilla.com/D46352 --HG-- extra : moz-landing-system : lando
2019-09-24 14:46:38 +00:00
security/manager/ssl/tests/unit,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
servo/components/style,
Bug 1562870 - generate conditioned profiles r=gbrown,bc,aerickson Generates "conditioned" profiles. Differential Revision: https://phabricator.services.mozilla.com/D38211 --HG-- extra : moz-landing-system : lando
2019-09-16 19:44:35 +00:00
testing/condprofile/condprof/android.py,
testing/condprofile/condprof/creator.py,
testing/condprofile/condprof/desktop.py,
testing/condprofile/condprof/runner.py,
testing/condprofile/condprof/scenarii/heavy.py,
Bug 1583197. Rename 'cold' conditioned profile to 'settled'. r=tarek Differential Revision: https://phabricator.services.mozilla.com/D54241 --HG-- rename : testing/condprofile/condprof/scenarii/cold.py => testing/condprofile/condprof/scenarii/settled.py extra : moz-landing-system : lando
2019-11-26 09:06:32 +00:00
testing/condprofile/condprof/scenarii/settled.py,
Bug 1597336 - add a fxa+sync scenario r=lina,markh This patch adds a scenario that ends up syncing the generated profile into FxSync. Differential Revision: https://phabricator.services.mozilla.com/D53789 --HG-- extra : moz-landing-system : lando
2020-02-23 23:17:23 +00:00
testing/condprofile/condprof/scenarii/synced.py
Bug 1581818 - Make sure client.py stays Python 2 compatible r=stephendonner We need to keep util.py python 2 compatible. The Python 3 helper class that's used to generate conditioned profiles is moved to its own module and we're reactivating util.py in flake8 Differential Revision: https://phabricator.services.mozilla.com/D46259 --HG-- extra : moz-landing-system : lando
2019-09-18 15:16:57 +00:00
testing/condprofile/condprof/helpers.py,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
testing/jsshell/benchmark.py,
testing/marionette/mach_commands.py,
testing/mozharness/docs,
testing/mozharness/examples,
testing/mozharness/external_tools,
testing/mozharness/mach_commands.py,
testing/mozharness/manifestparser,
testing/mozharness/mozprocess,
testing/mozharness/setup.py,
testing/parse_build_tests_ccov.py,
testing/runtimes/writeruntimes.py,
testing/tools/iceserver/iceserver.py,
testing/tools/websocketprocessbridge/websocketprocessbridge.py,
toolkit/components/featuregates,
toolkit/content/tests/chrome/file_about_networking_wsh.py,
Bug 1573566 - Move the real libxul definition in a subdirectory. r=froydnj The current setup, where gtest/libxul uses the static library in the same directory as the shared libxul, and somehow the backend ignores gkrust for gtest/libxul, is fragile. Differential Revision: https://phabricator.services.mozilla.com/D42246 --HG-- rename : toolkit/library/dependentlibs.py => toolkit/library/build/dependentlibs.py extra : moz-landing-system : lando
2019-08-16 21:44:10 +00:00
toolkit/library/build/dependentlibs.py,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
toolkit/locales/generate_update_locale.py,
toolkit/mozapps,
toolkit/moz.configure,
toolkit/nss.configure,
Bug 1805839 - Do not auto-format mako.XY template files r=sheehan,linter-reviewers,sylvestre The template syntax is incompatible with the host language syntax. Differential Revision: https://phabricator.services.mozilla.com/D164786
2023-01-02 11:46:53 +00:00
# mako files are not really python files
*.mako.py,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
# These paths are intentionally excluded (not necessarily for good reason).
Backed out changeset 4738f8111a82 (bug 1478338) for linting failure at builds/worker/checkouts/gecko/build/moz.configure/android-ndk.configure:8 on a CLOSED TREE
2018-07-25 13:56:19 +00:00
build/moz.configure/*.configure,
bug 1463425 - Add more directories to the flake8 list r=ahal MozReview-Commit-ID: JMZQonceq8W --HG-- extra : rebase_source : 1d8d0218f128dede197fa99e5cba1f0101a80384
2018-05-22 13:26:22 +00:00
build/pymake/,
Bug 1515746 - [flake8] Upgrade flake8 and dependencies, r=egao This bumps flake8 to version 3.7.5. This also ignores the new lint rules that were added in the new versions. These rules are de-marked via comment so we know that they should be enabled at some point (as opposed to the other rules that are (presumably) ignored intentionally. Differential Revision: https://phabricator.services.mozilla.com/D18353 --HG-- extra : moz-landing-system : lando
2019-02-07 20:16:45 +00:00
browser/extensions/mortar/ppapi/,
browser/moz.configure,
Bug 1468273 - Add closure-lib and intl/icu into the flake8 ignore list r=ahal MozReview-Commit-ID: A68UyXSHO9l --HG-- extra : rebase_source : 92c965aba41d31d47e437ef3e72070fe5d060338
2018-06-13 06:11:40 +00:00
dom/canvas/test/webgl-conf/checkout/closure-library/,
Bug 1468273 - autopep8 on gfx/ r=ted MozReview-Commit-ID: 8j8EU3E08GP --HG-- extra : rebase_source : 1f38ac0b7283505622cc95a120a0a0d4cea96803
2018-06-10 12:13:44 +00:00
editor/libeditor/tests/browserscope/,
Bug 1468273 - Add closure-lib and intl/icu into the flake8 ignore list r=ahal MozReview-Commit-ID: A68UyXSHO9l --HG-- extra : rebase_source : 92c965aba41d31d47e437ef3e72070fe5d060338
2018-06-13 06:11:40 +00:00
intl/icu/,
Bug 1508889 - Remove ipc/chromium from the lists of third-party code. r=Ehsan The ipc/chromium tree has been forked from its former upstream and heavily modified, so it's safe to reformat and lint. However, ipc/chromium/src/third_party (a copy of libevent) is following its upstream, so that should remain exempt. Differential Revision: https://phabricator.services.mozilla.com/D12498 --HG-- extra : moz-landing-system : lando
2018-11-21 03:11:37 +00:00
ipc/chromium/src/third_party/,
Bug 1515746 - [flake8] Upgrade flake8 and dependencies, r=egao This bumps flake8 to version 3.7.5. This also ignores the new lint rules that were added in the new versions. These rules are de-marked via comment so we know that they should be enabled at some point (as opposed to the other rules that are (presumably) ignored intentionally. Differential Revision: https://phabricator.services.mozilla.com/D18353 --HG-- extra : moz-landing-system : lando
2019-02-07 20:16:45 +00:00
js/*.configure,
Bug 1468273 - autopep8 on gfx/ r=ted MozReview-Commit-ID: 8j8EU3E08GP --HG-- extra : rebase_source : 1f38ac0b7283505622cc95a120a0a0d4cea96803
2018-06-10 12:13:44 +00:00
gfx/angle/,
gfx/harfbuzz,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
gfx/skia/,
Bug 1473308 - [flake8] Pass custom extension list into the flake8 binary r=marco We're supposed to be linting both .py and .configure files with flake8. However we never inform flake8 of this fact. So e.g running: ./mach lint -l flake8 mobile/android Will not lint mobile/android/gradle.configure. However since flake8 will run on a file regardless of its extension if you pass that file in directly, it means that running: ./mach lint -l flake8 mobile/android/gradle.configure *Will* cause the file to be linted (and subsequently fail). This fix makes sure that flake8 knows to look at .configure files in addition to .py. Since this means many .configure files around the tree will start getting linted for the first time, we need to exclude them until they can be fixed. Differential Revision: https://phabricator.services.mozilla.com/D1975 --HG-- extra : moz-landing-system : lando
2018-07-06 12:39:39 +00:00
memory/moz.configure,
mobile/android/*.configure,
Bug 1440421 part 1. Ignore activity-stream and node_modules for linting. r=ahal MozReview-Commit-ID: EEua2y790cW --HG-- extra : rebase_source : ce98dcbd82eb1f59594d6fe40e34648b7bd639de
2018-06-06 17:32:43 +00:00
node_modules,
Bug 1542963 - flake8 python/mozbuild/mozbuild r=glandium Lint python/mozbuild/{mozbuild,mozpack}. Differential Revision: https://phabricator.services.mozilla.com/D26639 --HG-- extra : moz-landing-system : lando
2019-05-28 14:21:43 +00:00
python/mozbuild/mozbuild/test/configure/data,
Bug 1464869 - Also ignore security/nss/ r=ahal MozReview-Commit-ID: 8m4CbrAaLHl --HG-- extra : rebase_source : 5643bb18d950e0d1010bea7f4a51f57195830aa9
2018-05-28 18:58:54 +00:00
security/nss/,
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
testing/marionette/harness/marionette_harness/runner/mixins,
testing/marionette/harness/marionette_harness/tests,
Bug 1654090 - Replace testing/mochitest/pywebsocket with pywebsocket3; r=jmaher Update testing/mochitest/pywebsocket with the latest version available: pywebsocket3 is python 3 compatible. This keeps the basic structure of the old pywebsocket, but changes the directory name to pywebsocket3 to reflect the project renaming. Differential Revision: https://phabricator.services.mozilla.com/D84455
2020-07-22 02:06:38 +00:00
testing/mochitest/pywebsocket3,
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
testing/mozharness/configs/test/test_malformed.py,
Bug 1622686 - Make testing/web-platform flake8 compliant. r=sylvestre,jgraham This also enables py3 linting for testing/web-platform. The external testing/web-platform/tests is excluded from linting. Differential Revision: https://phabricator.services.mozilla.com/D90744
2020-09-29 14:22:49 +00:00
testing/web-platform/tests,
Bug 1436802 - [lint] Add some tests for the flake8 linter integration r=andi This essentially tests tools/lint/python/flake8.py. Though it also adds a basic framework for testing all the other linters as well. Getting this added now will allow others to collaborate on adding more tests without needing to get to 100% coverage for all linters right off the bat. All python tests under tools/lint/test will run as part of the 'ml' task on Linux, and the build task on Windows (OSX coverage is currently missing for python tests). The flake8 linter currently has a bug where the 'exclude' argument is ignored. This is why we need to also exclude 'tools/lint/test/files' in topsrcdir/.flake8, even though it is already listed in the 'mach_commands.py'. Other linters shouldn't need to do this, the exclusion in 'mach_commands.py' should be good enough. See bug 1277851 for more details. MozReview-Commit-ID: 9ho8C83eeuj --HG-- extra : rebase_source : 0adb01aa81c06f26cb891ae9f92a17f8f9fdf111
2018-03-22 21:24:15 +00:00
tools/lint/test/files,
Bug 1519307 - Add a new project to build useful parts of breakpad independently. r=froydnj With `ac_add_options --enable-project=tools/crashreporter` in a mozconfig, `./mach build` builds minidump_stackwalk, dump_syms and fileid. One caveat is that due to limitation in how the build system works currently, it's cumbersome to keep dump_syms as a host program for Gecko, and to make it a target program for this project. For now, keep it as a host program. We're not going to use it on automation, but it's still convenient to have for quick local builds (I've had to resort to awful hacks downstream). Differential Revision: https://phabricator.services.mozilla.com/D16299 --HG-- extra : moz-landing-system : lando
2019-01-11 16:01:15 +00:00
tools/crashreporter/*.configure,
Bug 1367092 - [flake8] Use a blacklist instead of a whitelist, r=egao This ensures that the default for new python files is to be linted by flake8. Depends on D20496 Differential Revision: https://phabricator.services.mozilla.com/D20497 --HG-- extra : moz-landing-system : lando
2019-02-22 21:25:50 +00:00
.ycm_extra_conf.py,
Bug 1515746 - [flake8] Upgrade flake8 and dependencies, r=egao This bumps flake8 to version 3.7.5. This also ignores the new lint rules that were added in the new versions. These rules are de-marked via comment so we know that they should be enabled at some point (as opposed to the other rules that are (presumably) ignored intentionally. Differential Revision: https://phabricator.services.mozilla.com/D18353 --HG-- extra : moz-landing-system : lando
2019-02-07 20:16:45 +00:00
# See:
# - http://flake8.pycqa.org/en/latest/user/error-codes.html
# - http://pep8.readthedocs.io/en/latest/intro.html#configuration
ignore =
# These should be triaged and either fixed or moved to the list below.
Bug 1806068 - Ignore formatting issue in flake8 r=linter-reviewers,sylvestre DONTBUILD They are redundant with black formatting, so just ignore them. Differential Revision: https://phabricator.services.mozilla.com/D164884
2022-12-18 16:36:38 +00:00
W605, W606,
Bug 1515746 - [flake8] Upgrade flake8 and dependencies, r=egao This bumps flake8 to version 3.7.5. This also ignores the new lint rules that were added in the new versions. These rules are de-marked via comment so we know that they should be enabled at some point (as opposed to the other rules that are (presumably) ignored intentionally. Differential Revision: https://phabricator.services.mozilla.com/D18353 --HG-- extra : moz-landing-system : lando
2019-02-07 20:16:45 +00:00
# These are intentionally disabled (not necessarily for good reason).
# F723: syntax error in type comment
# text contains quotes which breaks our custom JSON formatter
Bug 1806068 - Ignore formatting issue in flake8 r=linter-reviewers,sylvestre DONTBUILD They are redundant with black formatting, so just ignore them. Differential Revision: https://phabricator.services.mozilla.com/D164884
2022-12-18 16:36:38 +00:00
F723, E704, E741,
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
Bug 1806068 - Ignore formatting issue in flake8 r=linter-reviewers,sylvestre DONTBUILD They are redundant with black formatting, so just ignore them. Differential Revision: https://phabricator.services.mozilla.com/D164884
2022-12-18 16:36:38 +00:00
# black is already in charge of formatting, no need to start a formatter
# battle here
E1, W1, E2, W2, E3, W3, E4, W4, E5, W5
Bug 1555560: Add support for black formatting with mozlint; r=sylvestre Differential Revision: https://phabricator.services.mozilla.com/D33125
2020-07-08 22:08:08 +00:00
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
per-file-ignores =
Bug 1587206 - [lint.flake8] Enable F811 across the tree, r=sylvestre This prevents redefinition of unused variables. Depends on D48610 Differential Revision: https://phabricator.services.mozilla.com/D48611 --HG-- extra : moz-landing-system : lando
2019-10-10 18:51:20 +00:00
# These paths are intentionally excluded.
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
ipc/ipdl/*: F403, F405
Bug 1587206 - [lint.flake8] Enable F811 across the tree, r=sylvestre This prevents redefinition of unused variables. Depends on D48610 Differential Revision: https://phabricator.services.mozilla.com/D48611 --HG-- extra : moz-landing-system : lando
2019-10-10 18:51:20 +00:00
layout/tools/reftest/selftest/conftest.py: F811
Bug 1542963 - Fix most flake8 issues in python/mozbuild/mozbuild/* r=firefox-build-system-reviewers,chmanchester Leaving one fix for an explicit review. Lint python/mozbuild/{mozbuild,mozpack}. r=#build Differential Revision: https://phabricator.services.mozilla.com/D26641 --HG-- extra : moz-landing-system : lando
2019-05-28 14:22:20 +00:00
# cpp_eclipse has a lot of multi-line embedded XML which exceeds line length
python/mozbuild/mozbuild/backend/cpp_eclipse.py: E501
Bug 1367092 - [flake8] Use per-file-ignores to skip __init__.py files under testing/marionette and testing/firefox-ui, r=ato This allows us to only skip the "unused import" config in these files rather than the entire thing. This also removes the only two uses of "**" in the exclusion rules which made things a bit simpler for me later on in the series. Differential Revision: https://phabricator.services.mozilla.com/D20493 --HG-- extra : moz-landing-system : lando
2019-02-22 14:37:55 +00:00
testing/firefox-ui/**/__init__.py: F401
testing/marionette/**/__init__.py: F401
Bug 1587206 - [lint.flake8] Enable F811 across the tree, r=sylvestre This prevents redefinition of unused variables. Depends on D48610 Differential Revision: https://phabricator.services.mozilla.com/D48611 --HG-- extra : moz-landing-system : lando
2019-10-10 18:51:20 +00:00
testing/mochitest/tests/python/conftest.py: F811
Bug 1583353 - [manifestparser] Support manifests in the 'pathprefix' filter, r=egao Allows 'paths' passed into the pathprefix filter to be manifests. Any path that ends with '.ini' is considered a manifest. Depends on D51899 Differential Revision: https://phabricator.services.mozilla.com/D51900 --HG-- extra : moz-landing-system : lando
2019-11-08 18:58:09 +00:00
testing/mozbase/manifestparser/tests/test_filters.py: E731
Bug 1671422 - [mozlog] Add group durations to errorsummary.log, r=marco Durations are measured in ms, and are computed by subtracting the timestamp of the first 'test_start' action from the last 'test_end' for each manifest. Differential Revision: https://phabricator.services.mozilla.com/D94178
2020-10-21 18:23:19 +00:00
testing/mozbase/mozlog/tests/test_formatters.py: E501
Bug 1515746 - [flake8] Unsupport subdir .flake8 files and use new 'per-file-ignores' config instead, r=egao This removes all .flake8 files except for the one at the root of the repo. Instead we use the new 'per-file-ignores' config introduced in 3.7. To ignore specific errors in a subdirectory, add a line like this to the root .flake8: [per-file-ignores] path/to/subdir/*: E100, F200, ... The reasons for this change are: 1. Unblock flake8 blacklist (bug 1367092). 2. Simplify configuration and code. 3. Encourage more consistent styling. 4. Improve performance. 5. Greater editor consistency. Differential Revision: https://phabricator.services.mozilla.com/D18354 --HG-- extra : moz-landing-system : lando
2019-02-07 20:17:02 +00:00
testing/mozharness/configs/*: E124, E127, E128, E131, E231, E261, E265, E266, E501, W391
Bug 1567642 - [lint.flake8] Skip remaining flake8 under Python 3 lint issues r=gbrown Depends on D45417 Differential Revision: https://phabricator.services.mozilla.com/D46352 --HG-- extra : moz-landing-system : lando
2019-09-24 14:46:38 +00:00
# These paths contain Python-2 only syntax which cause errors since flake8
# is run with Python 3.
Bug 1790708 - Clean up some per-file flake8 ignores that are no longer needed. r=linter-reviewers,sylvestre DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D157261
2022-09-14 06:17:51 +00:00
build/compare-mozconfig/compare-mozconfigs.py: F821
build/midl.py: F821
build/pgo/genpgocert.py: F821
config/MozZipFile.py: F821
config/check_source_count.py: F821
config/tests/unitMozZipFile.py: F821
Bug 1587206 - [lint.flake8] Enable F633 across the tree, r=sylvestre This prevents '>>' with print statements. Since this syntax is still valid in Python 2, I exluded dirs that contain this error rather than fix them. Depends on D48609 Differential Revision: https://phabricator.services.mozilla.com/D48610 --HG-- extra : moz-landing-system : lando
2019-10-10 18:50:52 +00:00
ipc/pull-chromium.py: F633
Bug 1790708 - Clean up some per-file flake8 ignores that are no longer needed. r=linter-reviewers,sylvestre DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D157261
2022-09-14 06:17:51 +00:00
js/src/**: F633, F821
python/mozbuild/mozbuild/action/dump_env.py: F821
python/mozbuild/mozbuild/dotproperties.py: F821
python/mozbuild/mozbuild/testing.py: F821
python/mozbuild/mozbuild/util.py: F821
testing/mozharness/mozharness/mozilla/testing/android.py: F821
testing/mochitest/runtests.py: F821
Bug 1642121 - Remove directory dependence of GDB init files. r=nalexander I wrote this patch to address two problems: 1. if I do `mach run` from a directory other than $topsrcdir, $objdir, or $objdir/dist/bin, then .gdbinit will not be loaded. 2. Debugging the firefox binary will never load the JS prettyprinters in any case. I believe this patch fixes other problems as well, such as .gdbinit_python not being found, and the gdbpp pretty-printers not getting loaded in various situations. The main changes of this patch are: 1. Move .gdbinit into build/ (and $objdir/build/) to delay it from getting loaded until the search path is configured. 2. Move libxul.so-gdb.py into the correct directory. 3. Use either libxul.so-gdb.py or js-gdb.py to configure the correct search path then load .gdbinit, and have .gdbinit load all of the pretty-printers (Gecko and JS). 4. Use a single preprocessed file to configure the source directory. Use relative paths within the objdir for everything else. Differential Revision: https://phabricator.services.mozilla.com/D77589
2020-06-16 15:45:43 +00:00
builtins =
# For GDB extensions
gdb
Reference in New Issue Copy Permalink