Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: 🛠 add upper boundary to Markupsafe to avoid breaking changes from unreleased v3 #63

Closed
wants to merge 2 commits into from

Conversation

wolfskaempf
Copy link

v3.0 of Markupsafe is described as unreleased and not stable, yet it is already on pypi. To avoid using it prematurely, we need to add this upper boundary.

…rom unreleased v3

v3.0 of Markupsafe is described as unreleased and not stable, yet it is already on pypi.
To avoid using it prematurely, we need to add this upper boundary.
@wolfskaempf
Copy link
Author

@pelme Some checks are failing on python3.13, I need to check whether this is related to the older version of Markupsafe or not.

@pelme
Copy link
Owner

pelme commented Oct 14, 2024

It seems like 3.0.1 is released but not updated on the docs site: pallets/markupsafe#474

So I think 3.x should be a fine release, will look into why it is failing with htpy!

@pelme
Copy link
Owner

pelme commented Oct 14, 2024

Jinja also uses markupsafe>2.0 so I think people would have a ton of breakages if it was a bad release!

Also: I will switch CI to use 3.13 final now that it is out.

@pelme
Copy link
Owner

pelme commented Oct 14, 2024

Switching to the Python 3.13 release (not release candidate) causes all tests to pass (see #45).

@wolfskaempf @nip3o can you provide some more details on what is wrong with markupsafe 3.x? All internal htpy tests are passing and they are quite extensive when it comes to checking integration with Django templates and when things are expected to be escaped or not.

@wolfskaempf
Copy link
Author

Now that it is clear that 3.0.1 is considered stable, the approach should not be to pin the version to <3. Therefore I will close this PR and instead open an issue with more details about the breakage after some further investigation.

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