Skip to content

Conversation

lthor
Copy link
Contributor

@lthor lthor commented Apr 22, 2025

Add an option to discard whitespace before the xml tag when reading from a stream.

Also updated a bad error message.

@lthor lthor requested a review from dgud April 22, 2025 12:06
Copy link
Contributor

github-actions bot commented Apr 22, 2025

CT Test Results

    2 files     24 suites   8m 1s ⏱️
2 165 tests 2 157 ✅  8 💤 0 ❌
3 983 runs  3 948 ✅ 35 💤 0 ❌

Results for commit 63e6d3e.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@lthor lthor self-assigned this Apr 22, 2025
@lthor lthor changed the base branch from master to maint April 22, 2025 12:31
@@ -0,0 +1,16 @@

Unknown functions:
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this file meant to be added?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, hmm wonder how I got that in. Always add files manually :).

@@ -0,0 +1,38 @@
<?xml version="1.0"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a license header o this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok.

@IngelaAndin IngelaAndin added the team:PS Assigned to OTP team PS label Apr 23, 2025
@lthor lthor added the testing currently being tested, tag is used by OTP internal CI label Apr 23, 2025
@lthor lthor force-pushed the lars/discard-space-before-xml-tag-27/OTP-19602 branch 2 times, most recently from c6ec2a0 to 8541180 Compare April 23, 2025 11:01
dgud
dgud previously approved these changes Apr 28, 2025
Add an option to discard whitespace before the xml tag
when reading from a stream.

Also updated a bad error message.
@lthor lthor force-pushed the lars/discard-space-before-xml-tag-27/OTP-19602 branch from 8541180 to 63e6d3e Compare May 6, 2025 12:41
@lthor lthor merged commit 9fa1c2b into erlang:maint May 7, 2025
21 checks passed
@lthor lthor removed the testing currently being tested, tag is used by OTP internal CI label May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:PS Assigned to OTP team PS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants