Page MenuHomeSoftware Heritage

Reformat docstring for max line length
ClosedPublic

Authored by twitu on Jun 27 2019, 6:23 AM.

Details

Summary

In reference to T1836, however all violations cannot be removed

Diff Detail

Repository
rDMOD Data model
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6527
Build 9094: tox-on-jenkinsJenkins
Build 9093: arc lint + arc unit

Event Timeline

twitu retitled this revision from Reformat docstring for max line length In reference to T1836, however all violations cannot be removed to Reformat docstring for max line length.Jun 27 2019, 6:30 AM
twitu edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jun 27 2019, 11:00 AM

You should be able to remove the # noqa flag at the end of the docstring now.

What do you mean by "all violations cannot be removed"?

This revision now requires changes to proceed.Jun 27 2019, 11:01 AM
swh/model/identifiers.py
662–663

this one is just exceeding 80 characters, it would mean moving the comment to the next line, which defeats the purpose of the line comment.

swh/model/identifiers.py
662–663

It should be fine on the next line.

Reformat docstring for max line length

This revision is now accepted and ready to land.Jun 27 2019, 6:44 PM
This revision was automatically updated to reflect the committed changes.