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

Merge 2.15.x up into 2.16.x #10841

Merged
merged 8 commits into from
Jul 12, 2023
Merged

Merge 2.15.x up into 2.16.x #10841

merged 8 commits into from
Jul 12, 2023

Conversation

greg0ire
Copy link
Member

No description provided.

greg0ire and others added 8 commits June 27, 2023 08:37

Verified

This commit was signed with the committer’s verified signature.
morozov Sergei Morozov
A lot of our tests mention it, but I do not think it is important to the
test. Maybe it was a way to have more efficient tests? Most times,
removing the hint does not affect the test outcome.

Verified

This commit was signed with the committer’s verified signature.
morozov Sergei Morozov
The way we have our docs, the sidebar is a separate document and as
such, needs a title. Let us prepend a dummy title until the guides-cli
provides a way to ignore that particular error.

Verified

This commit was signed with the committer’s verified signature.
morozov Sergei Morozov
In 7fa3e6e, a global search and replace
was used for http and https.
This broke the documentation examples in that as soon as you turn on XSD
validation, it will fail because the namespace in the XML file does not
match the ones defined in the XSD file, which do not exhibit the https.

Note that this is not a security concern, because these URIs are
not meant to be actually resolved, but to serve as a unique identifier
for the namespace in which we define our elements.

Verified

This commit was signed with the committer’s verified signature.
morozov Sergei Morozov
Match namespace in XML file with namespace in XSD file

Verified

This commit was signed with the committer’s verified signature.
morozov Sergei Morozov
Resort on Query::HINT_FORCE_PARTIAL_LOAD less
This was never meant to be under version control. Did not spot it in the
diff.

Closes #10836
@greg0ire greg0ire merged commit e781639 into 2.16.x Jul 12, 2023
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.

None yet

1 participant