Skip to content

fix(slug): correct circled characters - #147

Open
awais-ur-rehman wants to merge 1 commit into
pid:masterfrom
awais-ur-rehman:feat/circled-character-transliteration
Open

awais-ur-rehman wants to merge 1 commit into
pid:masterfrom
awais-ur-rehman:feat/circled-character-transliteration

Conversation

@awais-ur-rehman

Copy link
Copy Markdown

Summary

Circled ⓥ and ⓦ currently transliterate to the wrong letters, while ⑲ and ⑳ both become 18. Correct these four mappings in the source and the tracked browser bundle so a title such as Version ⓥ Week ⓦ Part ⑲ of ⑳ produces version-v-week-w-part-19-of-20.

Fixes #144.

Breaking Changes

None.

Testing

  • Added coverage for the complete circled lowercase/uppercase alphabet, maintainCase, circled numbers 0–20, a title slug, and browser-bundle consistency.
  • New regressions fail against the original source (56 passing, 6 failing).
  • npm test: 162 passing on Node 24.13.0.
  • node_modules/.bin/jshint lib/speakingurl.js test/test-symbols.js passes.
  • Actual npm pack --ignore-scripts output passes the title-slug check through both its Node entry point and browser bundle.
  • git diff --check passes.

This branch has not been deployed

No deployments
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.

ⓦ and ⓥ are mixed up and other mistakes

1 participant