Skip to content

chore: reformat ARB files to 4-space indent#7139

Merged
beastoin merged 1 commit into
mainfrom
fix/arb-indent-reformat
May 3, 2026
Merged

chore: reformat ARB files to 4-space indent#7139
beastoin merged 1 commit into
mainfrom
fix/arb-indent-reformat

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

@beastoin beastoin commented May 3, 2026

Summary

  • Reapply 4-space indentation to all 48 ARB localization files using jq --indent 4
  • These files were reverted to 2-space indent by recent commits (Apr 26-27) that were made without the pre-commit hook running

Context

The repo's scripts/pre-commit hook enforces 4-space indent on ARB files. Recent l10n contributions landed without this hook active, causing all ARB files to revert to 2-space formatting.

@aaravgarg @krushnarout — please make sure you have the pre-commit hook installed to keep the project standardized:

ln -s -f ../../scripts/pre-commit .git/hooks/pre-commit

Test plan

by AI for @beastoin

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 3, 2026

Greptile Summary

This PR reformats all 48 ARB localization files from 2-space to 4-space JSON indentation to match the enforcement policy in scripts/pre-commit. The change is purely cosmetic — the equal insertion/deletion counts (260,540 each) confirm no content was modified.

Confidence Score: 5/5

Safe to merge — purely cosmetic whitespace change with no logic or content modifications.

All 48 files show equal insertions and deletions (260,540 each), confirming this is a whitespace-only reformat from 2-space to 4-space indentation. The pre-commit hook in scripts/pre-commit explicitly enforces this standard via jq --indent 4. No ARB string values, keys, or metadata were altered.

No files require special attention.

Important Files Changed

Filename Overview
app/lib/l10n/app_ar.arb Reformatted from 2-space to 4-space indentation; no content changes.
app/lib/l10n/app_be.arb Reformatted from 2-space to 4-space indentation; no content changes (one of the larger files with ~10k entries).
app/lib/l10n/app_zh.arb Reformatted from 2-space to 4-space indentation; no content changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Contributor stages ARB file] --> B{pre-commit hook active?}
    B -- No --> C[2-space indent committed\nto main branch]
    B -- Yes --> D[jq --indent 4 reformats file]
    D --> E[4-space indent committed]
    C --> F[Formatting drift accumulates\nacross 48 ARB files]
    F --> G[This PR: bulk reformat\nvia jq --indent 4]
    G --> E
Loading

Reviews (1): Last reviewed commit: "chore: reformat ARB files to 4-space ind..." | Re-trigger Greptile

Reapply 4-space indentation via `jq --indent 4` to all 49 ARB
localization files. Recent commits (Apr 26-27) added ARB changes
without the pre-commit hook running, reverting formatting back to
2-space indent.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@beastoin beastoin force-pushed the fix/arb-indent-reformat branch from b3bd5f5 to 014c051 Compare May 3, 2026 07:56
@beastoin beastoin merged commit 54d9926 into main May 3, 2026
2 checks passed
@beastoin beastoin deleted the fix/arb-indent-reformat branch May 3, 2026 07:59
@beastoin
Copy link
Copy Markdown
Collaborator Author

beastoin commented May 3, 2026

lgtm

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