Skip to content

fix(de): revive #442 — relative units, ordinals, ago/later - #673

Open
Tzahreddin wants to merge 2 commits into
wanasit:masterfrom
Tzahreddin:revive/de-enhance-442
Open

Tzahreddin wants to merge 2 commits into
wanasit:masterfrom
Tzahreddin:revive/de-enhance-442

Conversation

@Tzahreddin

Copy link
Copy Markdown

Summary

Revival of @georgd’s work in #442 (Enhance de), rebased conceptually onto current master APIs (reverseDuration, no dayjs).

  • Fix vorletzte / übernächste (previously misparsed as letzte / nächste via bare vor + leftover stem)
  • Suffix modifiers früher / später (and synonyms)
  • Richer NUMBER_PATTERN (halb / viertel / dreiviertel / ein paar)
  • Ordinal day names for little-endian months (am dritten Oktober)
  • Day abbreviations d / tg, std.
  • Add de_time_units_ago / de_time_units_later tests
  • Removed commented nested-ago and strict-mode blocks per review on #442

Related

Test plan

  • npm test -- --testPathPattern='test/de/' (32 tests)
  • Full suite green via husky pre-commit (759 tests)

Disclosure: This pull request (title, description, and code changes in this contribution) was created and written with AI assistance.

Made with Cursor

Port Georg Duffner's Enhance-de work onto current master APIs:
- fix vorletzte/übernächste (was misparsed as letzte/nächste)
- suffix modifiers früher/später; richer NUMBER_PATTERN (halb/viertel/paar)
- ordinal day names for little-endian months; day abbr d/tg, std.
- add de_time_units_ago/later tests; drop commented nested/strict blocks per review

Addresses review notes on wanasit#442.

Note: This commit message and the accompanying PR were written with AI assistance.
Co-authored-by: Cursor <cursoragent@cursor.com>
@Tzahreddin

Copy link
Copy Markdown
Author

Follow-up planned (separate PR): ISO calendar weeks (KW / Kalenderwoche) and calendar week windows — related to #577 and the “week after next” family in #438.


Disclosure: This comment was written with AI assistance.

Note: This commit message was written with AI assistance.
Co-authored-by: Cursor <cursoragent@cursor.com>

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.

1 participant