Skip to content

Commit 7b303f5

Browse files
feanilclaude
andcommitted
style: fix pylint 4.x line-too-long violations
pylint was upgraded from 3.x to 4.x by make upgrade, which now enforces the 120-char line limit more strictly. - cms/envs/devstack.py, production.py: split long lines in triple-quoted string literals (Markdown text, so Python newlines become spaces — no content change) - test_containers.py: add pylint disable/enable around OLX assertion block where XML attribute strings cannot be split Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
1 parent a60e099 commit 7b303f5

3 files changed

Lines changed: 10 additions & 4 deletions

File tree

cms/envs/devstack.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -344,9 +344,11 @@ def should_show_debug_toolbar(request): # lint-amnesty, pylint: disable=missing
344344
access token endpoint: `{LMS_ROOT_URL}/oauth2/access_token`.
345345
Please see separately provided documentation.
346346
\n - How to test: You must be logged in as course author for whatever course you want to test with.
347-
You can use the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/) to "Try out" the API with your test course. To do this, you must select the "Local" server.
347+
You can use the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/) to "Try out" the API
348+
with your test course. To do this, you must select the "Local" server.
348349
\n - Public vs. Local servers: The "Public" server is where you can reach the API externally. The "Local" server is
349-
for development with a local edx-platform version, and for use via the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/).
350+
for development with a local edx-platform version, and for use via the
351+
[Swagger UI](https://{CMS_BASE}/authoring-api/ui/).
350352
\n - Swaggerfile: [Download link](https://{CMS_BASE}/authoring-api/schema/)''',
351353
'VERSION': '0.1.0',
352354
'SERVE_INCLUDE_SCHEMA': False,

cms/envs/production.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -409,9 +409,11 @@ def get_env_setting(setting):
409409
access token endpoint: `{LMS_ROOT_URL}/oauth2/access_token`.
410410
Please see separately provided documentation.
411411
\n - How to test: You must be logged in as course author for whatever course you want to test with.
412-
You can use the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/) to "Try out" the API with your test course. To do this, you must select the "Local" server.
412+
You can use the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/) to "Try out" the API
413+
with your test course. To do this, you must select the "Local" server.
413414
\n - Public vs. Local servers: The "Public" server is where you can reach the API externally. The "Local" server is
414-
for development with a local edx-platform version, and for use via the [Swagger UI](https://{CMS_BASE}/authoring-api/ui/).
415+
for development with a local edx-platform version, and for use via the
416+
[Swagger UI](https://{CMS_BASE}/authoring-api/ui/).
415417
\n - Swaggerfile: [Download link](https://{CMS_BASE}/authoring-api/schema/)''',
416418
'VERSION': '0.1.0',
417419
'SERVE_INCLUDE_SCHEMA': False,

openedx/core/djangoapps/content_libraries/tests/test_containers.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -840,6 +840,7 @@ def test_copy_container(self) -> None:
840840
# Test the actual OLX in the clipboard:
841841
olx_data = staging_api.get_staged_content_olx(clipboard_data.content.id)
842842
assert olx_data is not None
843+
# pylint: disable=line-too-long
843844
assert olx_data == textwrap.dedent(f"""\
844845
<chapter copied_from_block="{self.section_with_subsections["id"]}" copied_from_version="2" display_name="Section with subsections">
845846
<sequential copied_from_block="{self.subsection["id"]}" copied_from_version="1" display_name="Subsection Alpha"/>
@@ -858,6 +859,7 @@ def test_copy_container(self) -> None:
858859
<sequential copied_from_block="{self.subsection_3["id"]}" copied_from_version="1" display_name="Test Subsection 3"/>
859860
</chapter>
860861
""")
862+
# pylint: enable=line-too-long
861863

862864
def test_publish_subsection(self) -> None:
863865
"""

0 commit comments

Comments
 (0)