jqueguiner / num2words2 / 25211153053

95%

master: 94%

push

github

fix(aur): preserve trailing newline + strip CRLF in SSH key (#109)

* fix(tests): correct stale Arabic e2e expected output

Test #14 in tests/data/e2e_test_sentences.csv expected the ASCII comma
plus a stray double space (`تسعة وتسعون  , تسع`), but the converter
correctly emits the Arabic comma U+060C (`،`) — matching the comma
already present in the input sentence. Update the expected string to
match the correct Arabic-comma output.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* fix(lint): clear flake8 + isort backlog blocking CI

- Drop unused `cents_mode = None` local in num2words2/__init__.py
  (F841): the variable was assigned but never read.
- Remove the duplicate "Version information" try/except block in
  num2words2/__init__.py introduced by an earlier merge.
- Move the `from .grouping import group_digits` import to the import
  block proper (isort --float-to-top).
- Drop a leftover blank line in num2words2/lang_SR_LATN.py between
  `from .lang_SR import Num2Word_SR` and the data tables (isort).
- Add the missing two blank lines between `def`s in
  tests/test_v108_features.py (E302).
- Insert a single blank line after deferred `from num2words2 import
  num2words` in tests/lang/test_es.py, tests/lang/test_en.py,
  tests/lang/test_pt_BR.py (isort).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* fix(aur): preserve trailing newline + strip CRLF in SSH key

The v1.0.9 AUR publish failed with:
    Load key "/home/runner/.ssh/aur_key": error in libcrypto
    aur@aur.archlinux.org: Permission denied (publickey).

Cause: `printf '%s' "$AUR_SSH_KEY"` deliberately strips any trailing
newline, but OpenSSH requires a final LF on the key file or it can't
parse it (ed25519 in particular surfaces this as a libcrypto error).

Fix: switch to `printf '%s\n' | tr -d '\r'`, which both (a) guarantees
the trailing LF whether or not the secret was stored with one and
(b) strips any CRLF if the secret was pasted from... (continued)

7907 of 8678 branches covered (91.12%)

Branch coverage included in aggregate %.

41622 of 43363 relevant lines covered (95.99%)

5.76 hits per line