Skip to content

Conversation

EPAUL04
Copy link
Collaborator

@EPAUL04 EPAUL04 commented May 4, 2025

No description provided.

@EPAUL04 EPAUL04 requested a review from mauraoleary May 4, 2025 17:52
Copy link
Collaborator

@mauraoleary mauraoleary left a comment

Choose a reason for hiding this comment

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

Looks good to me - @jonorthwash?

@mauraoleary mauraoleary requested a review from jonorthwash July 16, 2025 21:19
Copy link
Member

@jonorthwash jonorthwash left a comment

Choose a reason for hiding this comment

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

I only notice a few things:

  • Dir/Lr should be Dir/LR — the case matters
  • The "apostrophe" character should be glottal stop ʼ, not smart quote . It looks like this was changed even for the preexisting entries?

@jonorthwash
Copy link
Member

Also:

  • It looks like (some of?) the preexisting words were removed? These should always be retained, and interleaved (alphabetically) with the additions.
  • In one spot two columns run together: riverbank"bare. In such cases, we need at least one space between the columns, even if it means one column doesn't line up perfectly with the others.

@EPAUL04
Copy link
Collaborator Author

EPAUL04 commented Jul 28, 2025

I’m having difficulties fixing the smart quote issue, as the apostrophe character that appears when I type a glottal stop seems to be the smart quote, even in VSCode.

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.

3 participants