Skip to content

fix(l10n): new translations#908

Merged
TobiTenno merged 15 commits into
masterfrom
l10n
Jul 3, 2026
Merged

fix(l10n): new translations#908
TobiTenno merged 15 commits into
masterfrom
l10n

Conversation

@wfcd-bot-boi

@wfcd-bot-boi wfcd-bot-boi commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Reformatted JSON data files across multiple language locales to improve consistency in formatting and indentation.
    • Updated whitespace/serialization for arcane entries and tutorial objects without changing their contents (fields like regex, name, effect, rarity, location, thumbnail, info, and url).

@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@TobiTenno, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 57 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5a542e15-900e-4f71-bfe1-e41fbf9c4133

📥 Commits

Reviewing files that changed from the base of the PR and between 34ea32e and e1b598b.

📒 Files selected for processing (1)
  • biome.json
📝 Walkthrough

Walkthrough

Whitespace and indentation were normalized across arcanes.json and tutorials.json for seven locales: cs, it, ko, pl, pt, sr, and tr. No keys, values, ordering, or JSON structure changed.

Changes

JSON Whitespace Reformatting

Layer / File(s) Summary
Arcanes JSON reformatting (all locales)
data/cs/arcanes.json, data/it/arcanes.json, data/ko/arcanes.json, data/pl/arcanes.json, data/pt/arcanes.json, data/sr/arcanes.json, data/tr/arcanes.json
Indentation and whitespace were normalized across all arcane entry arrays; regex, name, effect, rarity, location, thumbnail, and info values are unchanged.
Tutorials JSON reformatting (all locales)
data/cs/tutorials.json, data/it/tutorials.json, data/ko/tutorials.json, data/pl/tutorials.json, data/pt/tutorials.json, data/sr/tutorials.json, data/tr/tutorials.json
Indentation and whitespace were normalized for the three tutorial entries per locale; regex, name, and url values are unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: TobiTenno

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title suggests new translation content, but the PR only reformats existing l10n JSON files without changing text values. Rename it to reflect the actual change, e.g. "fix(l10n): reformat localization JSON files".
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch l10n

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (6)
data/ko/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/ko/tutorials.json` around lines 1 - 18, The file data/ko/tutorials.json
has formatting that does not comply with the project's formatting standards, as
indicated by the CI lint failure. Run the project's code formatter on this file
to automatically reformat it according to the project's whitespace and style
conventions. This will ensure the file passes the Actions/Lint CI checks.

Source: Pipeline failures

data/pt/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/pt/tutorials.json` around lines 1 - 18, The file data/pt/tutorials.json
has formatting that does not match the project's formatter standards and is
causing CI lint checks to fail. Run the project's formatter tool on this file to
automatically apply the correct whitespace and formatting conventions. This will
ensure the file passes the Actions / Lint check before the pull request can be
merged.

Source: Pipeline failures

data/sr/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/sr/tutorials.json` around lines 1 - 18, The tutorials.json file has
whitespace formatting that does not comply with the project's formatter
standards and is causing the Lint CI check to fail. Run the project's formatter
on the data/sr/tutorials.json file to apply the correct formatting, ensuring the
file passes the linting checks before merging.

Source: Pipeline failures

data/tr/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/tr/tutorials.json` around lines 1 - 18, The file data/tr/tutorials.json
has formatting that does not match the project's code style standards and is
causing the linting check to fail. Run the project's configured formatter (check
the project configuration for the formatter tool being used, such as prettier or
similar) on this file to automatically reformat the whitespace and indentation
to comply with the project standards. Once formatted, the file should pass the
Actions / Lint CI check.

Source: Pipeline failures

data/it/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/it/tutorials.json` around lines 1 - 18, The tutorials.json file has
whitespace formatting that does not match the project's formatter standards,
causing the Lint CI check to fail. Run the project's code formatter on the
data/it/tutorials.json file to automatically reformat the JSON with the correct
indentation and spacing according to project standards. This will resolve the CI
Lint action failure.

Source: Pipeline failures

data/pl/tutorials.json (1)

1-18: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Reformat this file with the project formatter.

CI already reports this file would be rewritten, so the current whitespace still fails Actions / Lint. Please apply the formatter output before merging.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/pl/tutorials.json` around lines 1 - 18, The tutorials.json file contains
improperly formatted JSON that is failing the lint checks. The file has entries
with fields like regex, name, and url that need to be reformatted according to
the project's formatter standards. Run the project's formatter tool on the
tutorials.json file to automatically fix the whitespace and formatting issues,
ensuring it passes the CI linting checks before merging.

Source: Pipeline failures

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@data/ko/arcanes.json`:
- Around line 2-460: The arcanes.json file is not formatted according to the
repository's expected formatting standards, which is causing the CI lint check
to fail. Run the repository's JSON formatter tool on the data/ko/arcanes.json
file to ensure the whitespace, indentation, and overall formatting matches the
repository's configuration exactly. This must be done before the changes will
pass CI validation.

---

Outside diff comments:
In `@data/it/tutorials.json`:
- Around line 1-18: The tutorials.json file has whitespace formatting that does
not match the project's formatter standards, causing the Lint CI check to fail.
Run the project's code formatter on the data/it/tutorials.json file to
automatically reformat the JSON with the correct indentation and spacing
according to project standards. This will resolve the CI Lint action failure.

In `@data/ko/tutorials.json`:
- Around line 1-18: The file data/ko/tutorials.json has formatting that does not
comply with the project's formatting standards, as indicated by the CI lint
failure. Run the project's code formatter on this file to automatically reformat
it according to the project's whitespace and style conventions. This will ensure
the file passes the Actions/Lint CI checks.

In `@data/pl/tutorials.json`:
- Around line 1-18: The tutorials.json file contains improperly formatted JSON
that is failing the lint checks. The file has entries with fields like regex,
name, and url that need to be reformatted according to the project's formatter
standards. Run the project's formatter tool on the tutorials.json file to
automatically fix the whitespace and formatting issues, ensuring it passes the
CI linting checks before merging.

In `@data/pt/tutorials.json`:
- Around line 1-18: The file data/pt/tutorials.json has formatting that does not
match the project's formatter standards and is causing CI lint checks to fail.
Run the project's formatter tool on this file to automatically apply the correct
whitespace and formatting conventions. This will ensure the file passes the
Actions / Lint check before the pull request can be merged.

In `@data/sr/tutorials.json`:
- Around line 1-18: The tutorials.json file has whitespace formatting that does
not comply with the project's formatter standards and is causing the Lint CI
check to fail. Run the project's formatter on the data/sr/tutorials.json file to
apply the correct formatting, ensuring the file passes the linting checks before
merging.

In `@data/tr/tutorials.json`:
- Around line 1-18: The file data/tr/tutorials.json has formatting that does not
match the project's code style standards and is causing the linting check to
fail. Run the project's configured formatter (check the project configuration
for the formatter tool being used, such as prettier or similar) on this file to
automatically reformat the whitespace and indentation to comply with the project
standards. Once formatted, the file should pass the Actions / Lint CI check.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2d7a2093-1148-4699-957f-4e714c862d5f

📥 Commits

Reviewing files that changed from the base of the PR and between 4c99e55 and 92b7e07.

📒 Files selected for processing (14)
  • data/cs/arcanes.json
  • data/cs/tutorials.json
  • data/it/arcanes.json
  • data/it/tutorials.json
  • data/ko/arcanes.json
  • data/ko/tutorials.json
  • data/pl/arcanes.json
  • data/pl/tutorials.json
  • data/pt/arcanes.json
  • data/pt/tutorials.json
  • data/sr/arcanes.json
  • data/sr/tutorials.json
  • data/tr/arcanes.json
  • data/tr/tutorials.json

Comment thread data/ko/arcanes.json Outdated
@TobiTenno TobiTenno enabled auto-merge (squash) July 3, 2026 21:51
@mergify

mergify Bot commented Jul 3, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@TobiTenno TobiTenno merged commit 9ec95d7 into master Jul 3, 2026
12 checks passed
@TobiTenno TobiTenno deleted the l10n branch July 3, 2026 21:52
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 3.16.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants