Skip to content

Conversation

flavorjones
Copy link
Member

What problem is this PR intended to solve?

Related to USN-3947-1 and USN-3947-2.

Addresses #1892

Have you included adequate test coverage?

No. Notably the upstream patch also has not added test coverage to libxslt. Imagine me making a judgmental face right now and you've probably nailed it.

Does this change affect the C or the Java implementations?

This affects the C implementation, and only when installing with the vendored version of libxslt; but not the Java implementation.

related to USN-3947-1 and USN-3947-2.

addresses #1892
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit fe034ae and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 93.5%.

View more on Code Climate.

@flavorjones flavorjones merged commit d31f742 into master Apr 22, 2019
@flavorjones flavorjones deleted the 1892-libxslt-patch-for-usn-3947 branch April 22, 2019 16:02
@flavorjones flavorjones added this to the v1.10.x patch releases milestone Apr 22, 2019
@flavorjones flavorjones added the backport Backport of a PR to the current release branch label Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport of a PR to the current release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant