Skip to content

Conversation

henryiii
Copy link
Contributor

@henryiii henryiii commented Oct 7, 2024

Adding testing for 3.13 in CI.

@henryiii
Copy link
Contributor Author

henryiii commented Oct 7, 2024

Stuck on cloudtoken, which is a Poetry project and puts tight upper caps on everything (including python<4, which is just annoying), so Collecting lxml==4.9.2 (from cloudtoken==2.1.18) breaks trying to compile old lxml.

Up to that point, had this:

-cloudtoken==2.1.0
+cloudtoken==2.1.18
 ipython==7.16.1
 isort==5.6.4
-jaraco-clipboard==2.0.1
+jaraco-clipboard==3.1.0

If someone wants to open an issue there, go ahead, I need to remember how to log into bitbucket. :)

@Gitznik
Copy link
Contributor

Gitznik commented Oct 10, 2024

cloudtoken has released 2.1.19, which removes the hard cap for the lxml version. Maybe that fixes your issue? I couldn't replicate the behavior of the pipeline locally for some reason, so I can't verify the fix.

@huxuan
Copy link
Member

huxuan commented Oct 10, 2024

Maybe we also need to delete the github action cache: https://github.com/pypa/pipx/actions/caches

@henryiii
Copy link
Contributor Author

I've cleared the cache and retrying.

@henryiii
Copy link
Contributor Author

Ahh, forgot to bump. 🤦

@henryiii
Copy link
Contributor Author

Don't see it? https://pypi.org/project/cloudtoken/

@Gitznik
Copy link
Contributor

Gitznik commented Oct 10, 2024

Don't see it? https://pypi.org/project/cloudtoken/

Right sorry I looked at the repo and saw they bumped the version 2 days ago, but I guess they have not actually gone through and do a release.

@huxuan
Copy link
Member

huxuan commented Jan 29, 2025

Maybe we should skip the tests related to cloudtoken temporarily? Any other suggestions?

@dukecat0
Copy link
Member

+1 on this. We can add it back later if a new version is released on PyPI.

@uranusjr
Copy link
Member

The entire cloudtoken repo is now gone. Maybe we should switch to another package instead.

@henryiii
Copy link
Contributor Author

Sounds good to me. :)

henryiii and others added 6 commits May 21, 2025 00:40
@henryiii henryiii force-pushed the henryiii-patch-1 branch from 102b59e to c760043 Compare May 21, 2025 04:41
@henryiii
Copy link
Contributor Author

henryiii commented May 21, 2025

I don't think modern cloudtoken actually has the issue that 0.1.84 seems to have been added for from the changelog (infinite recursion) https://inspector.pypi.io/project/cloudtoken/0.1.84/packages/1c/62/ae6d69f1f2bd624e459d115883d0f5f882a716a876b89bbfb6eb190fc641/cloudtoken-0.1.84-py3-none-any.whl/cloudtoken-0.1.84.dist-info/METADATA. I'm trying that version, as it was a lot simpler, might pass on newer Pythons.

@dukecat0 dukecat0 mentioned this pull request Jun 16, 2025
1 task
@dukecat0
Copy link
Member

Closing in favour of #1647

@dukecat0 dukecat0 closed this Sep 29, 2025
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.

5 participants