Skip to content

feat(mention_utils): add support for group-mention m tags in rust#928

Merged
synoet merged 2 commits intomainfrom
synoet/md-group-support-rust
Jan 12, 2026
Merged

feat(mention_utils): add support for group-mention m tags in rust#928
synoet merged 2 commits intomainfrom
synoet/md-group-support-rust

Conversation

@synoet
Copy link
Copy Markdown
Contributor

@synoet synoet commented Jan 12, 2026

Forgot to add parsing for m-group-mention tags in the main pr.

@synoet synoet requested a review from a team as a code owner January 12, 2026 18:18
@synoet synoet changed the title feat(mention_utils): add support for grou-mention m tags in rust feat(mention_utils): add support for group-mention m tags in rust Jan 12, 2026
use crate::parse::{
ParsedContactMention, ParsedDateMention, ParsedDocumentMention, ParsedLink, ParsedUserMention,
ParsedXmlText, TextSegment, XmlTag,
ParsedContactMention, ParsedDateMention, ParsedDocumentMention, ParsedGroupMention, ParsedLink,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

just a side note but we really need to standardize on tests.rs vs test.rs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

lol yeah

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Jan 12, 2026


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


@claude
Copy link
Copy Markdown
Contributor

claude bot commented Jan 12, 2026


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


@synoet synoet merged commit 618e3a9 into main Jan 12, 2026
41 checks passed
@synoet synoet deleted the synoet/md-group-support-rust branch January 12, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants