Page MenuHomeSoftware Heritage

docs: Ask readers to install swh.model[cli] to fully use swh-identify
ClosedPublic

Authored by vlorentz on Apr 8 2021, 7:27 PM.

Details

Summary

Otherwise, they will get an error asking them to install Click
(or Dulwich if Click installed and they use -t snapshot)

Resolves T3226

Diff Detail

Repository
rDMOD Data model
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 D5469 (id=19542)

Could not rebase; Attempt merge onto eeedac7af7...

Updating eeedac7..383e002
Fast-forward
 docs/persistent-identifiers.rst |  2 +-
 swh/model/cli.py                | 18 ++++++++++++++++--
 tox.ini                         | 21 ++++++++++++++++++++-
 3 files changed, 37 insertions(+), 4 deletions(-)
Changes applied before test
commit 383e00280c3ceaa1094f82b30aa231cde225d850
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:26:53 2021 +0200

    docs: Ask readers to install swh.model[cli] to fully use swh-identify
    
    Otherwise, they will get an error asking them to install Click
    (or Dulwich if Click installed and they use -t snapshot)

commit f2e323830841c2aa5f1cdada1781e6f483830109
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:18:10 2021 +0200

    tox: Check swh-identify can run even if Dulwich isn't installed

commit 3e4bf9cb108d30fe2a727acdef69c61fa6b1f1d0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 18:55:01 2021 +0200

    swh-identify: Hide tracebacks if Click or Dulwich is not installed
    
    And show nice human-readable errors instead

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

This revision is now accepted and ready to land.Apr 8 2021, 7:32 PM

Build is green

Patch application report for D5469 (id=19579)

Could not rebase; Attempt merge onto eeedac7af7...

Updating eeedac7..984ccc9
Fast-forward
 docs/persistent-identifiers.rst |  2 +-
 pytest.ini                      |  1 +
 requirements-test.txt           |  1 -
 setup.py                        |  1 +
 swh/model/cli.py                | 18 ++++++++++++++++--
 swh/model/tests/test_cli.py     | 15 +++++++++++++++
 tox.ini                         | 22 +++++++++++-----------
 7 files changed, 45 insertions(+), 15 deletions(-)
Changes applied before test
commit 984ccc9c64ca2ffddebd4d7900def888bc6565d0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:26:53 2021 +0200

    docs: Ask readers to install swh.model[cli] to fully use swh-identify
    
    Otherwise, they will get an error asking them to install Click
    (or Dulwich if Click installed and they use -t snapshot)

commit e7bbfd04d7940996938e680b2d808cc4fd10a2e7
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:18:10 2021 +0200

    tox: Check swh-identify can run even if Dulwich isn't installed

commit c62f13fd55739b73245cbfa75f0325df4de12957
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 18:55:01 2021 +0200

    swh-identify: Hide tracebacks if Click or Dulwich is not installed
    
    And show nice human-readable errors instead

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

Build is green

Patch application report for D5469 (id=19601)

Could not rebase; Attempt merge onto eeedac7af7...

Updating eeedac7..f2dba17
Fast-forward
 docs/persistent-identifiers.rst |  2 +-
 pytest.ini                      |  1 +
 requirements-test.txt           |  1 -
 setup.py                        |  1 +
 swh/model/cli.py                | 18 ++++++++++++++++--
 swh/model/tests/test_cli.py     | 15 +++++++++++++++
 tox.ini                         | 22 +++++++++++-----------
 7 files changed, 45 insertions(+), 15 deletions(-)
Changes applied before test
commit f2dba177adf000b5bf855e98d16afc762bd9b0e0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:26:53 2021 +0200

    docs: Ask readers to install swh.model[cli] to fully use swh-identify
    
    Otherwise, they will get an error asking them to install Click
    (or Dulwich if Click installed and they use -t snapshot)

commit 27a05d687f845029e5f8b35a84a63bb7f0e877ff
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 19:18:10 2021 +0200

    tox: Check swh-identify can run even if Dulwich isn't installed

commit c62f13fd55739b73245cbfa75f0325df4de12957
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 8 18:55:01 2021 +0200

    swh-identify: Hide tracebacks if Click or Dulwich is not installed
    
    And show nice human-readable errors instead

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