Skip to content

Conversation

@josh-fell
Copy link
Contributor

Related: PyGithub/PyGithub#2436

Starting with v1.58 of PyGithub, serialization with pickle of Repository, NamedUser, Issue, and other github.Github.GithubObjects is no longer possible out of the box. Users won't be able to pass these objects in XCom if desired.

@josh-fell
Copy link
Contributor Author

josh-fell commented Feb 23, 2023

I'm not entirely sure if this is too aggressive, but I definitely welcome feedback.

Is it worth coming up with a serializer for the provider to handle the stateful object?

@potiuk
Copy link
Member

potiuk commented Feb 23, 2023

It looks like - from your issue @josh-fell that they are eyeying fixing it. So I would rather use != 1.58 now.

@josh-fell
Copy link
Contributor Author

It looks like - from your issue @josh-fell that they are eyeying fixing it. So I would rather use != 1.58 now.

Sounds good, thanks @potiuk. I honestly wasn't sure how to interpret the response. I'll update the constraint.

@josh-fell
Copy link
Contributor Author

josh-fell commented Feb 27, 2023

Actually it looks like a PR was opened several hours ago to fix it. 🎉

Starting with v1.58 of `PyGithub`, serialization with `pickle` of `Repository`, `NamedUser`, `Issue`, and other `github.Github.GithubObject`s is no longer possible out of the box. Users won't be able to pass these objects in XCom if desired when using the GitHhub provider.
@josh-fell josh-fell changed the title Set upper limit for PyGithub package Exclude v1.58 of PyGithub package in GitHub provider Feb 27, 2023
@josh-fell josh-fell marked this pull request as ready for review February 27, 2023 15:15
@josh-fell josh-fell merged commit a3d0e79 into apache:main Feb 27, 2023
@josh-fell josh-fell deleted the pygithub branch February 27, 2023 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants