Page MenuHomeSoftware Heritage
Differential D7645

Move GitHubSession from github/lister.py to github/utils.py
ClosedPublic
Actions

Authored by vlorentz on Apr 25 2022, 3:45 PM.

Details

Reviewers
olasd
Group Reviewers
Reviewers
Commits
rDLS2d04244cc950: Move GitHubSession from github/lister.py to github/utils.py
Summary

So it can be reused by other packages without importing lister.py itself

Addresses D7630#199321

Diff Detail

Repository
rDLS Listers
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz created this revision.Apr 25 2022, 3:45 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptApr 25 2022, 3:45 PM
swh-public-ci added a comment.Apr 25 2022, 3:49 PM

Build has FAILED

Patch application report for D7645 (id=27666)

Could not rebase; Attempt merge onto 2fa9f0abd2...

Updating 2fa9f0a..3937b6c
Fast-forward
 swh/lister/github/lister.py            | 168 ++------------------------------
 swh/lister/github/tests/test_lister.py |   9 +-
 swh/lister/github/utils.py             | 170 +++++++++++++++++++++++++++++++++
 3 files changed, 183 insertions(+), 164 deletions(-)
 create mode 100644 swh/lister/github/utils.py
Changes applied before test
commit 3937b6cab5bfd62d2795c79ca09f014928070c65
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit f91caedbb2ed5eca2572b7b5ebe1413692496f60
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

commit d0924f39d04e7b7128c55a116280e2c6b6ec5a00
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:32:45 2022 +0200

    github: Remove dead code
    
    Authentication is handled directly in the session

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/495/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/495/console

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 25 2022, 3:49 PM
Harbormaster failed remote builds in B28723: Diff 27666!
vlorentz updated this revision to Diff 27667.Apr 25 2022, 3:49 PM

fix tests

swh-public-ci added a comment.Apr 25 2022, 3:53 PM

Build has FAILED

Patch application report for D7645 (id=27667)

Rebasing onto 2fa9f0abd2...

Current branch diff-target is up to date.
Changes applied before test
commit 82a3717ec79e3890f5793ac2dafb740dda7989b1
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit f91caedbb2ed5eca2572b7b5ebe1413692496f60
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

commit d0924f39d04e7b7128c55a116280e2c6b6ec5a00
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:32:45 2022 +0200

    github: Remove dead code
    
    Authentication is handled directly in the session

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/496/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/496/console

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 25 2022, 3:53 PM
Harbormaster failed remote builds in B28724: Diff 27667!
vlorentz updated this revision to Diff 27668.Apr 25 2022, 3:54 PM

again

vlorentz mentioned this in D7630: github: Refactor rate-limiting out of the GitHubLister class.Apr 25 2022, 3:55 PM
swh-public-ci added a comment.Apr 25 2022, 3:58 PM

Build is green

Patch application report for D7645 (id=27668)

Could not rebase; Attempt merge onto 2fa9f0abd2...

Updating 2fa9f0a..2a99323
Fast-forward
 swh/lister/github/lister.py            | 168 ++------------------------------
 swh/lister/github/tests/test_lister.py |  15 +--
 swh/lister/github/utils.py             | 170 +++++++++++++++++++++++++++++++++
 3 files changed, 186 insertions(+), 167 deletions(-)
 create mode 100644 swh/lister/github/utils.py
Changes applied before test
commit 2a993230e6547ad19ce90f11e09c1a187c6c60a3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit f91caedbb2ed5eca2572b7b5ebe1413692496f60
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

commit d0924f39d04e7b7128c55a116280e2c6b6ec5a00
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:32:45 2022 +0200

    github: Remove dead code
    
    Authentication is handled directly in the session

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

Harbormaster completed remote builds in B28725: Diff 27668.Apr 25 2022, 3:58 PM
vlorentz requested review of this revision.Apr 25 2022, 3:58 PM
olasd accepted this revision.Apr 25 2022, 5:00 PM
olasd added a subscriber: olasd.

Thanks!

This revision is now accepted and ready to land.Apr 25 2022, 5:00 PM
vlorentz updated this revision to Diff 27699.Apr 26 2022, 11:08 AM

rebase

swh-public-ci added a comment.Apr 26 2022, 11:12 AM

Build is green

Patch application report for D7645 (id=27699)

Could not rebase; Attempt merge onto 334c54091e...

Updating 334c540..2d04244
Fast-forward
 swh/lister/github/lister.py            | 160 ++-----------------------------
 swh/lister/github/tests/test_lister.py |  15 +--
 swh/lister/github/utils.py             | 170 +++++++++++++++++++++++++++++++++
 3 files changed, 186 insertions(+), 159 deletions(-)
 create mode 100644 swh/lister/github/utils.py
Changes applied before test
commit 2d04244cc950bfad254b7337449d2fe20e36f610
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit 9ee4a99f152d9d6cbe5a6a09895ce9384660be46
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

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

Harbormaster completed remote builds in B28756: Diff 27699.Apr 26 2022, 11:12 AM
Closed by commit rDLS2d04244cc950: Move GitHubSession from github/lister.py to github/utils.py (authored by vlorentz). · Explain WhyApr 26 2022, 11:28 AM
This revision was automatically updated to reflect the committed changes.
vlorentz added a commit: rDLS2d04244cc950: Move GitHubSession from github/lister.py to github/utils.py.

Revision Contents
Changeset List

PathSize
swh/
lister/
github/
162 lines
tests/
9 lines
170 lines

Diff 27704

View Options

swh/lister/github/lister.py

Loading...
View Options

swh/lister/github/tests/test_lister.py

Loading...
View Options

swh/lister/github/utils.py

Loading...
About · Developer information · Legal