Skip to content

docs(i18n): clarify locale setup and lunaria file requirement#443

Merged
danielroe merged 1 commit intomainfrom
docs/i18n-contributing-guide
Jan 30, 2026
Merged

docs(i18n): clarify locale setup and lunaria file requirement#443
danielroe merged 1 commit intomainfrom
docs/i18n-contributing-guide

Conversation

@danielroe
Copy link
Copy Markdown
Member

Summary

Clarifies the i18n locale setup instructions in CONTRIBUTING.md to prevent common mistakes:

  • Simplified the main flow for adding a single locale (most common case)
  • Added explicit step to copy translation file to lunaria/files/ with important note that it must be committed
  • Moved country variants documentation to a separate "advanced" section
  • Removed confusing/redundant instructions

This addresses issues seen in PRs #433 and #437 where contributors didn't know about the lunaria file requirement.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 30, 2026 9:14pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Jan 30, 2026 9:14pm
npmx-lunaria Ignored Ignored Jan 30, 2026 9:14pm

Request Review

Copy link
Copy Markdown
Member

@userquin userquin left a comment

Choose a reason for hiding this comment

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

LGTM

@danielroe danielroe enabled auto-merge January 30, 2026 21:18
@danielroe danielroe disabled auto-merge January 30, 2026 21:18
@danielroe danielroe merged commit b26b3c2 into main Jan 30, 2026
16 checks passed
@danielroe danielroe deleted the docs/i18n-contributing-guide branch January 30, 2026 21:18
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