Skip to content

Conversation

@ChrisJefferson
Copy link

The version of pyyaml we use was failing due to a cython related issue. After a quick google, the easiest option seems to be to go to an even older version -- that fixes CI on all versions we support, and pyyaml is only used for testing, so it doesn't matter it's old and insecure.

@ChrisJefferson ChrisJefferson mentioned this pull request Jan 16, 2024
@noahp
Copy link
Member

noahp commented Dec 1, 2025

@ChrisJefferson I think we're properly pinned now, should be OK to close this one:
https://github.com/dtrx-py/dtrx/blob/master/pyproject.toml#L38

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