Skip to content

Add curly apostrophe check#10285

Open
albin-mullvad wants to merge 5 commits intomainfrom
add-curly-apostrophe-check-droid-1716
Open

Add curly apostrophe check#10285
albin-mullvad wants to merge 5 commits intomainfrom
add-curly-apostrophe-check-droid-1716

Conversation

@albin-mullvad
Copy link
Copy Markdown
Contributor

@albin-mullvad albin-mullvad commented Apr 23, 2026

What

This PR adds checks that ensure we use consistent curly apostrophes (or rather avoids straight ones) throughout our English user-facing strings. It also updates a few incorrect strings.

Why

We've had regressions where curly and straight apostrophes are mixed many times.

How

Adds a script and workflow to ensure we use curly apostrophes.

TODO

  • Decide and configure code ownership rule fo scripts/check-apostrophe-style.
  • Fix lack of multi-line support for Android/linux.

This change is Reviewable

@linear
Copy link
Copy Markdown

linear Bot commented Apr 23, 2026

@albin-mullvad albin-mullvad force-pushed the add-curly-apostrophe-check-droid-1716 branch from 66ced24 to 117956b Compare April 23, 2026 11:45
@albin-mullvad albin-mullvad added the Android Issues related to Android label Apr 23, 2026
@albin-mullvad albin-mullvad self-assigned this Apr 23, 2026
@albin-mullvad albin-mullvad marked this pull request as ready for review April 23, 2026 12:16
Copy link
Copy Markdown
Collaborator

@Rawa Rawa left a comment

Choose a reason for hiding this comment

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

@Rawa reviewed 29 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved.

@albin-mullvad albin-mullvad force-pushed the add-curly-apostrophe-check-droid-1716 branch from 8f3932f to e57906c Compare April 23, 2026 12:34
@albin-mullvad albin-mullvad requested a review from Rawa April 23, 2026 13:14
Copy link
Copy Markdown
Contributor

@rablador rablador left a comment

Choose a reason for hiding this comment

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

@rablador reviewed 29 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved.

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

Labels

Android Issues related to Android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants