Skip to content

Conversation

karlcow
Copy link
Member

@karlcow karlcow commented Jul 23, 2025

Editorial: Adds spaces in the UA pattern for Safari

This fixes the pattern for Safari UA string, which was missing spaces.

Fixes #279


Preview | Diff

@karlcow karlcow requested a review from miketaylr July 23, 2025 23:06
@annevk
Copy link
Member

annevk commented Jul 24, 2025

Can you please make sure the commit title/body follows the guidelines?

@karlcow karlcow changed the title fix #279. Adds spaces in the UA pattern for Safari Editorial: Adds spaces in the UA pattern for Safari Jul 24, 2025
@karlcow
Copy link
Member Author

karlcow commented Jul 24, 2025

Can you please make sure the commit title/body follows the guidelines?

is it in the shape expected by the guidelines?

Copy link
Member

@miketaylr miketaylr left a comment

Choose a reason for hiding this comment

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

LGTM - but can you remove "Editorial:" from the commit message? This is a bugfix, so it shouldn't have that.

@annevk annevk merged commit 5610f6a into main Jul 27, 2025
2 checks passed
@annevk annevk deleted the 279-fix-spaces-ua-pattern branch July 27, 2025 14:45
@karlcow
Copy link
Member Author

karlcow commented Jul 28, 2025

confused. in between @miketaylr and @annevk 😝

@annevk
Copy link
Member

annevk commented Jul 28, 2025

As per https://github.com/whatwg/meta/blob/main/COMMITTING.md#structure-and-conventions commit titles are not sentences and issues are to be referenced in the commit body. That's all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Safari generic UA pattern is not right
3 participants