Skip to content

data: Trim newlines from product_name#232

Merged
vnepogodin merged 1 commit intomasterfrom
fix/trim-product-name
Mar 24, 2026
Merged

data: Trim newlines from product_name#232
vnepogodin merged 1 commit intomasterfrom
fix/trim-product-name

Conversation

@1Naim
Copy link
Copy Markdown
Member

@1Naim 1Naim commented Mar 23, 2026

fs::read_to_string(path) returns product_name\n. This breaks pattern matching with string anchors fail as the pattern fails to match against the newline. Trim newlines from product_name so that matching patterns with string anchors work correctly.

@1Naim 1Naim requested a review from vnepogodin March 23, 2026 14:02
@1Naim 1Naim force-pushed the fix/trim-product-name branch from a497320 to b959f13 Compare March 23, 2026 14:05
fs::read_to_string(path) returns `product_name\n`. This breaks pattern
matching with string anchors fail as the pattern fails to match against
the newline. Trim newlines from product_name so that matching patterns
with string anchors work correctly.

Fixes: 7c30436 ("👷 support product name pattern match")
Signed-off-by: Eric Naim <dnaim@cachyos.org>
@1Naim 1Naim force-pushed the fix/trim-product-name branch from b959f13 to f794748 Compare March 23, 2026 14:17
Copy link
Copy Markdown
Member

@vnepogodin vnepogodin left a comment

Choose a reason for hiding this comment

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

fair

@vnepogodin vnepogodin merged commit 754c827 into master Mar 24, 2026
1 of 2 checks passed
@vnepogodin vnepogodin deleted the fix/trim-product-name branch March 24, 2026 13:28
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.

2 participants