Page MenuHomeSoftware Heritage

Pin black to the same version as the pre-commit hook
ClosedPublic

Authored by olasd on Sep 18 2020, 3:45 PM.

Details

Summary

Apparently, there's a new black release with Opinions on docstrings

Test Plan

tox -e black doesn't suggest changes to half the source tree

Diff Detail

Repository
rDCORE Foundations and core functionalities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D3991 (id=14069)

Could not rebase; Attempt merge onto 28d61c8cf1...

Updating 28d61c8..30be938
Fast-forward
 swh/core/db/db_utils.py | 26 ++++++++++++++------------
 tox.ini                 |  2 +-
 2 files changed, 15 insertions(+), 13 deletions(-)
Changes applied before test
commit 30be93846340bb289f9e780c5b0d6c50dfe8b506
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Fri Sep 18 15:42:24 2020 +0200

    Pin black to the same version as the pre-commit hook

commit b63d7f432caeb9d75da6151fb20f3764beec7f80
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Fri Sep 18 14:39:50 2020 +0200

    Fix license statements on db_utils.py
    
    When the code was migrated from swh.storage, some other ancillary functions were
    merged into this file. So bits of it are under the LGPLv3 (the code pulled from
    psycopg2) and bits are under the GPLv3 (the code we wrote).

See https://jenkins.softwareheritage.org/job/DCORE/job/tests-on-diff/64/ for more details.

ardumont added a subscriber: ardumont.

tox -e black doesn't suggest changes to half the source tree

yeah, we did that yesterday already...

so lol, that made me smile :) ¯\_(ツ)_/¯ ;)

This revision is now accepted and ready to land.Sep 18 2020, 3:46 PM