In reference to T1836
Details
Details
- Reviewers
- None
- Group Reviewers
Reviewers - Commits
- rDDEP216d0f74d8c3: Reformat docstrings for max line length
Diff Detail
Diff Detail
- Repository
- rDDEP Push deposit
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 6511 Build 9067: tox-on-jenkins Jenkins Build 9066: arc lint + arc unit
Event Timeline
Comment Actions
Build has FAILED
Link to build: https://jenkins.softwareheritage.org/job/DDEP/job/tox/166/
See console output for more information: https://jenkins.softwareheritage.org/job/DDEP/job/tox/166/console
Comment Actions
It is odd that py3 test cases are failing although I have only changed to formatting of the docstrings.
Comment Actions
These are not docstring.
A docstring is a string that is either at the very top of a function, or on the line right after a variable assignment.
What you changed are regular strings, which are assigned to a variable (eg. self.atom_error_with_decimal = b"""<content of the string>"""), so changing them changes the value of the variable.
Comment Actions
A source of confusion is that triple double quotes (""") are often used for docstrings, but what they actually represent is a multi-line strings. (And docstrings are a popular use of multi-line strings, hence the confusion)
Comment Actions
I will close this diff since no changes required for swh-deposit. With complex cases like this there is no way this process can be automated.