Skip to content

Fix italic markdown detection at line start #7

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.

Sign up for GitHub

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

Jump to bottom
Merged
JoshTheWanderer merged 2 commits into from
Jan 23, 2026
Merged

Fix italic markdown detection at line start #7

JoshTheWanderer merged 2 commits into from
Jan 23, 2026
+14 −1

Conversation

@JoshTheWanderer
Copy link
Member

@JoshTheWanderer JoshTheWanderer commented Jan 23, 2026

Summary

  • Fix italic pattern to allow *italic* at the start of a line/message
  • The previous pattern used (?<!^) which prevented matching at line start
  • New pattern uses lookarounds to avoid **bold** and * list items without excluding line starts
  • Add tests for italic at line start and multi-line italic

🤖 Generated with Claude Code

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 23, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@JoshTheWanderer JoshTheWanderer merged commit bc40617 into main Jan 23, 2026
12 checks passed
@JoshTheWanderer JoshTheWanderer deleted the fix/italic-and-bold branch January 23, 2026 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gavmck gavmck gavmck approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JoshTheWanderer @gavmck