Skip to content

Commit

Permalink
Enable --experimental-string-processing on most primer projects (#…
Browse files Browse the repository at this point in the history
…2184)

* Enable ` --experimental-string-processing` on all primer projects
- We want to make this default so need to test it more
- Fixed splat/star bug in extending black args for each project

* Disable sqlalchemy due to crash
  • Loading branch information
cooperlees committed May 5, 2021
1 parent 14c76e8 commit 07c8812
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 24 deletions.
2 changes: 1 addition & 1 deletion src/black_primer/lib.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ async def black_run(
"""Run Black and record failures"""
cmd = [str(which(BLACK_BINARY))]
if "cli_arguments" in project_config and project_config["cli_arguments"]:
cmd.extend(*project_config["cli_arguments"])
cmd.extend(project_config["cli_arguments"])
cmd.append("--check")
if no_diff:
cmd.append(".")
Expand Down
47 changes: 24 additions & 23 deletions src/black_primer/primer.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,28 +2,28 @@
"configuration_format_version": 20200509,
"projects": {
"aioexabgp": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": false,
"git_clone_url": "https://github.com/cooperlees/aioexabgp.git",
"long_checkout": false,
"py_versions": ["all"]
},
"attrs": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/python-attrs/attrs.git",
"long_checkout": false,
"py_versions": ["all"]
},
"bandersnatch": {
"cli_arguments": [],
"expect_formatting_changes": false,
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/pypa/bandersnatch.git",
"long_checkout": false,
"py_versions": ["all"]
},
"channels": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/django/channels.git",
"long_checkout": false,
Expand All @@ -32,107 +32,108 @@
"django": {
"disabled_reason": "black --check --diff returned 123 on tests_syntax_error.py",
"disabled": true,
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/django/django.git",
"long_checkout": false,
"py_versions": ["all"]
},
"flake8-bugbear": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": false,
"git_clone_url": "https://github.com/PyCQA/flake8-bugbear.git",
"long_checkout": false,
"py_versions": ["all"]
},
"hypothesis": {
"cli_arguments": [],
"expect_formatting_changes": false,
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/HypothesisWorks/hypothesis.git",
"long_checkout": false,
"py_versions": ["all"]
},
"pandas": {
"disabled_reason": "black-primer runs failing on Pandas - #2193",
"disabled": true,
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/pandas-dev/pandas.git",
"long_checkout": false,
"py_versions": ["all"]
},
"pillow": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/python-pillow/Pillow.git",
"long_checkout": false,
"py_versions": ["all"]
},
"poetry": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/python-poetry/poetry.git",
"long_checkout": false,
"py_versions": ["all"]
},
"pyanalyze": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": false,
"git_clone_url": "https://github.com/quora/pyanalyze.git",
"long_checkout": false,
"py_versions": ["all"]
},
"pyramid": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/Pylons/pyramid.git",
"long_checkout": false,
"py_versions": ["all"]
},
"ptr": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/facebookincubator/ptr.git",
"long_checkout": false,
"py_versions": ["all"]
},
"pytest": {
"cli_arguments": [],
"expect_formatting_changes": false,
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/pytest-dev/pytest.git",
"long_checkout": false,
"py_versions": ["all"]
},
"sqlalchemy": {
"no_cli_args_reason": "breaks black with new string parsing - #2188",
"cli_arguments": [],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/sqlalchemy/sqlalchemy.git",
"long_checkout": false,
"py_versions": ["all"]
},
"tox": {
"cli_arguments": [],
"expect_formatting_changes": false,
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/tox-dev/tox.git",
"long_checkout": false,
"py_versions": ["all"]
},
"typeshed": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/python/typeshed.git",
"long_checkout": false,
"py_versions": ["all"]
},
"virtualenv": {
"cli_arguments": [],
"expect_formatting_changes": false,
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/pypa/virtualenv.git",
"long_checkout": false,
"py_versions": ["all"]
},
"warehouse": {
"cli_arguments": [],
"cli_arguments": ["--experimental-string-processing"],
"expect_formatting_changes": true,
"git_clone_url": "https://github.com/pypa/warehouse.git",
"long_checkout": false,
Expand Down

0 comments on commit 07c8812

Please sign in to comment.