-
-
Notifications
You must be signed in to change notification settings - Fork 212
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.
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
Apply ruff/flake8-implicit-str-concat rule ISC001 #1044
Conversation
At first glance all occurences where intentionally line split and had gotten cobbled to a single line, i think multi line Strings are needed |
This is the result of applying black in #255, and Would you rather add |
I think real multi line Strings should be used |
I'll try to do that with |
what i meant was using tripple quote string, not disabling fmt |
80ddcad
to
65e102b
Compare
testing/test_regressions.py
Outdated
@@ -35,7 +35,8 @@ def test_pkginfo_noscmroot(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> N | |||
|
|||
tmp_path.joinpath(".git").mkdir() | |||
p.joinpath("setup.py").write_text( | |||
"from setuptools import setup;" 'setup(use_scm_version={"root": ".."})', | |||
'''from setuptools import setup; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to keep setup.py
as an one-liner?
from setuptools import setup;setup(use_scm_version={"root": ".."})
Or perhaps change as follows?
from setuptools import setup
setup(use_scm_version={"root": ".."})
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea
4b80b86
to
9ce7c1e
Compare
- Revert to multiline strings, as before pypa#255 / ac2fee8. - Guard the multiline strings from black (and now ruff) unfolding them back to a single line. https://docs.astral.sh/ruff/rules/single-line-implicit-string-concatenation/
thanks! |
https://docs.astral.sh/ruff/rules/single-line-implicit-string-concatenation/