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

Fix block parameter handling ruby #4093

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

tachyons
Copy link
Contributor

@tachyons tachyons commented Aug 18, 2024

Fix block parameter handling

Changes

in ruby block parameter can be enclosed with |. But ruby also has builtin operator |=. This PR ensure that |= is not marked as be the beginning of the block parameter.
Before:

Screenshot 2024-08-19 at 5 44 09 AM

After:

Screenshot 2024-08-19 at 5 44 43 AM

Checklist

  • Added markup tests, or they don't apply here because...
  • Updated the changelog at CHANGES.md

@joshgoebel
Copy link
Member

Awesome, just please add an entry to CHANGES.md.

in ruby block parameter can be enclosed with |. But ruby also has
builtin operator |=. This PR ensure that |= is not marked as be the
beginning of the block parameter.
@tachyons tachyons marked this pull request as ready for review August 19, 2024 00:33
@tachyons
Copy link
Contributor Author

@joshgoebel Done

Copy link

Build Size Report

Changes to minified artifacts in /build, after gzip compression.

2 files changed

Total change +9 B

View Changes
file base pr diff
es/languages/ruby.min.js 1.83 KB 1.84 KB +4 B
languages/ruby.min.js 1.83 KB 1.84 KB +5 B

@joshgoebel joshgoebel merged commit da79da6 into highlightjs:main Aug 19, 2024
19 checks passed
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.

3 participants