Skip to content

Match the whole LanguageData tree in the editorconfig byte-preserve rule#244

Merged
ptr727 merged 1 commit into
developfrom
feature/languagedata-recursive-glob
Jul 5, 2026
Merged

Match the whole LanguageData tree in the editorconfig byte-preserve rule#244
ptr727 merged 1 commit into
developfrom
feature/languagedata-recursive-glob

Conversation

@ptr727

@ptr727 ptr727 commented Jul 5, 2026

Copy link
Copy Markdown
Owner

The [LanguageData/*] section only matched files directly under LanguageData/, leaving the iso6392/, iso6393/, rfc5646/, and unm49/ subtrees without the no-normalization override. Use the recursive [LanguageData/**] glob so the override covers every downloaded data file, matching the * -text gitattributes contract.

Surfaced by the develop->main promotion review (#243).

Copilot AI review requested due to automatic review settings July 5, 2026 20:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s EditorConfig rules so the “byte-preserve” override applies to the entire downloaded language data tree, aligning editor behavior with the existing .gitattributes no-normalization contract for LanguageData/.

Changes:

  • Change the EditorConfig section header from [LanguageData/*] to [LanguageData/**] so subdirectories are included.
  • Ensure all downloaded data files under LanguageData/ avoid editor normalization (charset/EOL/final newline/trim).

The [LanguageData/*] section only matched files directly under LanguageData/, leaving the iso6392/, iso6393/, rfc5646/, and unm49/ subtrees without the no-normalization override. Use the recursive [LanguageData/**] glob so the override covers every downloaded data file, matching the * -text gitattributes contract.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ptr727 ptr727 force-pushed the feature/languagedata-recursive-glob branch from e47c262 to cb7689a Compare July 5, 2026 20:59
@ptr727 ptr727 requested a review from Copilot July 5, 2026 21:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@ptr727 ptr727 merged commit 701dd6a into develop Jul 5, 2026
11 checks passed
@ptr727 ptr727 deleted the feature/languagedata-recursive-glob branch July 5, 2026 21:09
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.

2 participants