Skip to content

GitHub Action for non-UTF-8 locales#7821

Open
aitap wants to merge 4 commits into
masterfrom
GHA-locales
Open

GitHub Action for non-UTF-8 locales#7821
aitap wants to merge 4 commits into
masterfrom
GHA-locales

Conversation

@aitap

@aitap aitap commented Jul 18, 2026

Copy link
Copy Markdown
Member

Following #7681 (comment): test data.table in the Latin-1 locale (where CE_NATIVE strings should be byte-to-byte equal to CE_LATIN1), GB18030 (which is fully Unicode-compatible, but the mapping from code points to byte sequences is very non-uniform), KOI8-R (which can represent some math symbols but not extended Latin or CJK).

aitap added 4 commits June 6, 2026 14:03
In a non-UTF-8 locale capable of representing a-umlaut, o-umlaut,
u-umlaut, encode from the locale encoding, not UTF-8.
@aitap
aitap requested a review from MichaelChirico as a code owner July 18, 2026 17:17
@aitap aitap changed the title GitHub Actions for non-UTF-8 locales GitHub Action for non-UTF-8 locales Jul 18, 2026
@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.01%. Comparing base (94649d5) to head (060f490).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7821   +/-   ##
=======================================
  Coverage   99.01%   99.01%           
=======================================
  Files          88       88           
  Lines       17234    17234           
=======================================
  Hits        17065    17065           
  Misses        169      169           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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