Skip to content

feat: sync feature set pygithub#60

Merged
coreycaverly merged 22 commits into
argoproj-labs:masterfrom
kaosx5s:sync-feature-set-pygithub
Feb 27, 2026
Merged

feat: sync feature set pygithub#60
coreycaverly merged 22 commits into
argoproj-labs:masterfrom
kaosx5s:sync-feature-set-pygithub

Conversation

@kaosx5s

@kaosx5s kaosx5s commented Feb 27, 2026

Copy link
Copy Markdown
Collaborator

Description

coreycaverly and others added 6 commits February 26, 2026 17:19
Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>
* Allow path define on get files (argoproj-labs#42)

* Add support for setting a path on get_files

Signed-off-by: kaosx5s <jeremy.xfives@gmail.com>

* fix repo test after target change in 3.6.0

Signed-off-by: kaosx5s <>

---------

Signed-off-by: kaosx5s <jeremy.xfives@gmail.com>
Signed-off-by: kaosx5s <>
Co-authored-by: kaosx5s <>
Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Removed transient line (argoproj-labs#50)

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Revving version for release (argoproj-labs#51)

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>
Co-authored-by: Corey Caverly <corey.caverly@autodesk.com>
Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Update docker-publish.yml (argoproj-labs#52)

* Update docker-publish.yml

Removing Dockerhub and just doing ghcr

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Revving for release

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Decided to remove and not just comment out

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

---------

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* retry on timeout (argoproj-labs#54)

* retry on timeout

Signed-off-by: zliu3 <zejun_liu@intuit.com>

* add unit test and throw exception on timeout

Signed-off-by: zliu3 <zejun_liu@intuit.com>

* update README.md

Signed-off-by: zliu3 <zejun_liu@intuit.com>

* add test to make sure retry on exception, add a good case

Signed-off-by: zliu3 <zejun_liu@intuit.com>

---------

Signed-off-by: zliu3 <zejun_liu@intuit.com>
Co-authored-by: zliu3 <zejun_liu@intuit.com>
Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* feat: drop support for older python versions, support up to 2.6.1 for pygithub (argoproj-labs#56)

* fix: drop support for outdated python

* fix: update pygithub to support 2.2.0 or higher

* fix: update tests for newer pytest

* fix: check pygithub 2.4.0 support

* fix: check pygithub 2.6.1 support

* fix: set lazy to be false by default for backwards compatibility

* fix: remove completed param for pygithub 2.6.0 support

* fix: add required requester for pygithub 2.6.0 support

* fix: fix param typo

* fix: make requester a magicmock

* fix: update tests for new lazy param

* fix: update lazy from str to bool

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* revving for release

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Revving python version for publish

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Updating readme to reflect new location of docker images

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

* Revving for new release

Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>

---------

Signed-off-by: kaosx5s <jeremy.xfives@gmail.com>
Signed-off-by: kaosx5s <>
Signed-off-by: Corey Caverly <corey.caverly@autodesk.com>
Signed-off-by: zliu3 <zejun_liu@intuit.com>
Co-authored-by: Jeremy <1028480+kaosx5s@users.noreply.github.com>
Co-authored-by: zejunintuit <zejunliuajk@gmail.com>
Co-authored-by: zliu3 <zejun_liu@intuit.com>

@coreycaverly coreycaverly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@coreycaverly coreycaverly merged commit 8c22fdd into argoproj-labs:master Feb 27, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants