Skip to content

otp: add bidi scanner#10920

Open
kikofernandez wants to merge 2 commits intoerlang:maintfrom
kikofernandez:kiko/bidi-detection
Open

otp: add bidi scanner#10920
kikofernandez wants to merge 2 commits intoerlang:maintfrom
kikofernandez:kiko/bidi-detection

Conversation

@kikofernandez
Copy link
Copy Markdown
Contributor

addition of bidi scanner and fix of invisible unicode from when
Erlang/OTP translated XML to markdown.

@kikofernandez kikofernandez self-assigned this Mar 25, 2026
@kikofernandez kikofernandez requested a review from u3s March 25, 2026 13:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 25, 2026

CT Test Results

    3 files    142 suites   49m 35s ⏱️
1 652 tests 1 596 ✅ 56 💤 0 ❌
2 374 runs  2 299 ✅ 75 💤 0 ❌

Results for commit 5cf8b4c.

♻️ 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

@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@kikofernandez kikofernandez force-pushed the kiko/bidi-detection branch 3 times, most recently from 8b6658d to c0d795a Compare March 25, 2026 14:31
addition of bidi scanner and fix of invisible unicode from when
Erlang/OTP translated XML to markdown.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The removed characters are SIX-PER-EM-SPACE:s (U+2006), that I would like to be kept as they were.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I got that wrong. They are ZERO WIDTH SPACE (U+200B)

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team:VM Assigned to OTP team VM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants