jaraco/inflect

number_to_words: support for custom comma-separators is broken/incomplete

Offen

#176 geöffnet am 13.02.2023

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Python (125 Forks)github user discovery
bughelp wanted

Repository-Metriken

Stars
 (1.082 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Discovered while refactoring the number_to_words method, as part of reducing complexity of the package in #174:

Using a custom comma argument to number_to_words appears to be supported based on the docstrings, and that would match the corresponding documentation for the EN::Inflect CPAN module that this library is based on.

However: in practice it seems that providing a comma argument (other than the default ,) does not work as expected.

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung von number to words in inflect/ init .py und konzentrieren Sie sich auf die Behandlung des Arguments comma. Vergleichen Sie mit der Dokumentation des EN::Inflect CPAN Moduls und stellen Sie sicher, dass das benutzerdefinierte Trennzeichen korrekt angewendet wird. Verfolgen Sie die Testfälle, um das erwartete Verhalten mit dem tatsächlichen zu vergleichen.
Tech Stack
python
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45