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

Exclude xmlsec version 1.3.14 #18

Merged
merged 5 commits into from
Jul 8, 2024
Merged

Exclude xmlsec version 1.3.14 #18

merged 5 commits into from
Jul 8, 2024

Conversation

pt2302
Copy link
Contributor

@pt2302 pt2302 commented Jul 8, 2024

What are the relevant tickets?

Related to xmlsec/python-xmlsec#314. This has notably caused problems in ocw-studio, requiring multiple manual reverts (e.g., mitodl/ocw-studio#2180, mitodl/ocw-studio#2235).

Description (What does it do?)

This PR excludes the xmlsec version 1.3.14 across all mitodl repositories for Renovate.

How can this be tested?

This can be tested by running renovate --dry-run in a relevant repository (such as ocw-studio) and verifying that the xmlsec version is not upgraded to 1.3.14.

@pt2302 pt2302 added the Needs Review An open Pull Request that is ready for review label Jul 8, 2024
},
{
"matchPackageNames": ["xmlsec"],
"allowedVersions": "<=1.3.13"
Copy link
Member

Choose a reason for hiding this comment

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

Since it looks like the problem is specifically with 1.3.14 I would argue that rather than setting the specifer to <=1.3.13 it should just be !/1\.3\.14/ so that we can continue to get updates for newer releases, but avoid this specific problematic version.

Copy link
Member

Choose a reason for hiding this comment

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

@pt2302 pt2302 changed the title Pin xmlsec version to 1.3.13 or earlier Exclude xmlsec version 1.3.14 Jul 8, 2024
@pt2302 pt2302 requested a review from blarghmatey July 8, 2024 13:56
@pt2302 pt2302 removed the Needs Review An open Pull Request that is ready for review label Jul 8, 2024
@pt2302 pt2302 merged commit f022b59 into main Jul 8, 2024
1 check passed
@pt2302 pt2302 deleted the pt/pin_xmlsec_version branch July 8, 2024 19:15
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