Skip to content

Conversation

@herver1971
Copy link

Adjusted the indentation and formatting of the attribute list in the TestCase docstring to improve readability and avoid Sphinx documentation build errors.

The attributes failureException, longMessage, and maxDiff are now more clearly presented with reduced indentation, making the list easier to read and avoiding issues with unexpected indentation in Sphinx.

No logic or functionality was modified; this change only affects the docstring for documentation purposes.

Adjusted the indentation and formatting of the attribute list in the TestCase docstring to improve readability and avoid Sphinx documentation build errors. 

The attributes `failureException`, `longMessage`, and `maxDiff` are now more clearly presented with reduced indentation, making the list easier to read and avoiding issues with unexpected indentation in Sphinx. 

No logic or functionality was modified; this change only affects the docstring for documentation purposes.
@ghost
Copy link

ghost commented Nov 3, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Nov 3, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@herver1971
Copy link
Author

Hi, @maintainers. Could you please add the "skip issue" label to this PR? I don’t have permissions to add labels myself. Thank you!

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modifying the start of the docstring is unnecessary, per PEP 257.

The indentation has not be changed, lines have been un-wrapped? That is wrong, and breaks PEP 8.

Closing as problems are introduced and there is no clear benefit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants