Skip to content

docs: add Spanish translation of CONTRIBUTING.md onboarding sections (fixes #125) - #162

Merged
ulises-jeremias merged 4 commits into
Create-Python-App:mainfrom
emberb170d:translate-contributing-es-2
Aug 17, 2026
Merged

docs: add Spanish translation of CONTRIBUTING.md onboarding sections (fixes #125)#162
ulises-jeremias merged 4 commits into
Create-Python-App:mainfrom
emberb170d:translate-contributing-es-2

Conversation

@emberb170d

@emberb170demberb170d commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Translates the CONTRIBUTING.md onboarding sections to Spanish as requested in issue #125. Covers: clone, local file:// test, PR expectations, link to AUTHORING.md. English version remains canonical.

Summary by CodeRabbit

  • Documentation

    • Added a Spanish contribution guide covering extensions, templates, metadata, commits, pull request checks, local testing, and support resources.
  • Template Improvements

    • Added consistent formatting settings across multiple starter templates.
    • Standardized Python version selection to Python 3.12 for supported templates.

@coderabbitai

coderabbitaiBot commented Aug 9, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

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

Next review available in:54 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

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 Plus

Run ID: 00581e08-a3e7-4d8f-97a4-7ef0d64f6bcd

📥 Commits

Reviewing files that changed from the base of the PR and between 517c48a and 07fe361.

📒 Files selected for processing (1)
  • templates/mlops-sklearn-starter/CONTRIBUTING.md
📝 Walkthrough

Walkthrough

Adds a Spanish contribution guide and standardizes editor settings across five templates. The templates also declare Python 3.12 through .python-version files.

Changes

Contributor guidance

Layer / File(s)Summary
Spanish contribution guide
CONTRIBUTING.es.md
Documents extension and template contributions, metadata requirements, conventional commits, pull request checks, local testing, support links, and the contribution workflow.

Template development defaults

Layer / File(s)Summary
Template formatting and Python defaults
templates/*/.editorconfig, templates/*/.python-version
Adds EditorConfig rules to five templates and declares Python 3.12 for the Celery worker, CLI starter, Django API, FastAPI starter, and MLOps starter templates.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

  • Create-Python-App/create-python-app, issue 265 — Both changes add Spanish contributor guidance in CONTRIBUTING.es.md.
  • Create-Python-App/cpa-templates, issue 117 — Both changes document the Spanish template contribution workflow.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title accurately describes the Spanish contribution guide added in the pull request, although it omits the template configuration changes.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitaicoderabbitaiBot 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: 3

🧹 Nitpick comments (1)
CONTRIBUTING.es.md (1)

48-50: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Keep one Spanish contribution guide.

Lines 7-46 already provide detailed instructions. Lines 52-73 repeat the same workflow. Remove one copy or make the second section a direct reference to the first. Otherwise, the two sections can diverge.

As per PR objectives: the English version remains canonical, so the Spanish content should have one maintainable summary.

🤖 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 `@CONTRIBUTING.es.md` around lines 48 - 50, Consolida el contenido duplicado de
CONTRIBUTING.es.md: conserva una única guía resumida en español y elimina la
repetición entre las secciones existentes, o convierte la segunda sección en una
referencia directa a la primera. Mantén CONTRIBUTING.md como versión canónica y
evita conservar dos flujos de contribución independientes.
🤖 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 `@CONTRIBUTING.es.md`:
- Around line 70-71: Update the “Prueba localmente” section in
CONTRIBUTING.es.md to include the canonical local file:// testing instructions,
including uv sync and uv run pytest, either directly or via a link to the exact
docs/TESTING.md section; do not leave it as a generic testing-guide reference.
In `@templates/celery-worker/.editorconfig`:
- Around line 11-12: Restore the missing EditorConfig selectors in all five
files: templates/celery-worker/.editorconfig (lines 11-12),
templates/cli-starter/.editorconfig (lines 11-12),
templates/django-api/.editorconfig (lines 11-12),
templates/fastapi-starter/.editorconfig (lines 11-12), and
templates/mlops-sklearn-starter/.editorconfig (lines 11-12). Add md to each
2-space selector and add a [Makefile] section setting indent_style = tab.
In `@templates/uv-workspace-starter/.editorconfig`:
- Around line 11-12: Add the repository’s Markdown and Makefile recipe selectors
to the EditorConfig alongside the existing YAML, JSON, and TOML section,
preserving the inherited formatting defaults despite root = true. Use the
repository root .editorconfig definitions for the exact selectors and
indentation settings.
---
Nitpick comments:
In `@CONTRIBUTING.es.md`:
- Around line 48-50: Consolida el contenido duplicado de CONTRIBUTING.es.md:
conserva una única guía resumida en español y elimina la repetición entre las
secciones existentes, o convierte la segunda sección en una referencia directa a
la primera. Mantén CONTRIBUTING.md como versión canónica y evita conservar dos
flujos de contribución independientes.
🪄 Autofix

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 Plus

Run ID: 65551ec3-beee-4347-875c-c91c892153e4

📥 Commits

Reviewing files that changed from the base of the PR and between 6986f9b and 517c48a.

📒 Files selected for processing (12)
  • CONTRIBUTING.es.md
  • templates/celery-worker/.editorconfig
  • templates/celery-worker/.python-version
  • templates/cli-starter/.editorconfig
  • templates/cli-starter/.python-version
  • templates/django-api/.editorconfig
  • templates/django-api/.python-version
  • templates/fastapi-starter/.editorconfig
  • templates/fastapi-starter/.python-version
  • templates/mlops-sklearn-starter/.editorconfig
  • templates/mlops-sklearn-starter/.python-version
  • templates/uv-workspace-starter/.editorconfig

Comment threadCONTRIBUTING.es.md
Comment on lines +70 to +71
6. **Prueba localmente**
- Consulta [docs/TESTING.md](./docs/TESTING.md) para guías de pruebas locales

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win

Add the local file:// testing instructions.

The Spanish guide does not mention file://, uv sync, or uv run pytest. Line 70 only delegates testing to docs/TESTING.md. Add the actual local testing steps or link directly to the exact canonical section.

As per PR objectives: the Spanish onboarding must cover local file:// testing.

🤖 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 `@CONTRIBUTING.es.md` around lines 70 - 71, Update the “Prueba localmente”
section in CONTRIBUTING.es.md to include the canonical local file:// testing
instructions, including uv sync and uv run pytest, either directly or via a link
to the exact docs/TESTING.md section; do not leave it as a generic testing-guide
reference.

Comment on lines +11 to +12
[*.{yml,yaml,json,toml}]
indent_size = 2 No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Restore the missing EditorConfig selectors in all five templates.

Each file sets root = true, so it does not inherit the repository root .editorconfig. Add Markdown to the 2-space selector and add the Makefile tab rule.

  • templates/celery-worker/.editorconfig#L11-L12: include md and add [Makefile] with indent_style = tab.
  • templates/cli-starter/.editorconfig#L11-L12: include md and add [Makefile] with indent_style = tab.
  • templates/django-api/.editorconfig#L11-L12: include md and add [Makefile] with indent_style = tab.
  • templates/fastapi-starter/.editorconfig#L11-L12: include md and add [Makefile] with indent_style = tab.
  • templates/mlops-sklearn-starter/.editorconfig#L11-L12: include md and add [Makefile] with indent_style = tab.
📍 Affects 5 files
  • templates/celery-worker/.editorconfig#L11-L12 (this comment)
  • templates/cli-starter/.editorconfig#L11-L12
  • templates/django-api/.editorconfig#L11-L12
  • templates/fastapi-starter/.editorconfig#L11-L12
  • templates/mlops-sklearn-starter/.editorconfig#L11-L12
🤖 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 `@templates/celery-worker/.editorconfig` around lines 11 - 12, Restore the
missing EditorConfig selectors in all five files:
templates/celery-worker/.editorconfig (lines 11-12),
templates/cli-starter/.editorconfig (lines 11-12),
templates/django-api/.editorconfig (lines 11-12),
templates/fastapi-starter/.editorconfig (lines 11-12), and
templates/mlops-sklearn-starter/.editorconfig (lines 11-12). Add md to each
2-space selector and add a [Makefile] section setting indent_style = tab.

Comment on lines +11 to +12
[*.{yml,yaml,json,toml}]
indent_size = 2 No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Preserve the complete EditorConfig contract.

Because root = true stops parent configuration inheritance, this file omits the repository root .editorconfig rules for Markdown indentation and Makefile recipes. Add both selectors so generated projects keep the repository formatting defaults.

🤖 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 `@templates/uv-workspace-starter/.editorconfig` around lines 11 - 12, Add the
repository’s Markdown and Makefile recipe selectors to the EditorConfig
alongside the existing YAML, JSON, and TOML section, preserving the inherited
formatting defaults despite root = true. Use the repository root .editorconfig
definitions for the exact selectors and indentation settings.

@ulises-jeremias

Copy link
Copy Markdown
Member

Spanish onboarding for #125 is welcome, but this PR is wider than the title and is not mergeable as-is:

  • It still includes the MLOps CONTRIBUTING rewrite (docs(mlops): add contributor checklist for first MLOps template PR #121) and the already-merged .editorconfig / .python-version files. Please keep only CONTRIBUTING.es.md.
  • The issue asked for a condensed Spanish guide that includes the local file:// / CPA_TEMPLATES_URL recipe, not a second full copy of CONTRIBUTING.md.
  • Small wording fixes: “commits convencionales”, “si admite/soporta múltiples tipos”, “La versión canónica es la inglesa.”
  • Add one line in English CONTRIBUTING.md pointing at CONTRIBUTING.es.md, and link Spanish readers to docs/AUTHORING.es.md.

Please rebase onto current main after that so L1 actually runs.

@ulises-jeremias
ulises-jeremias merged commit b53e01c into Create-Python-App:mainAug 17, 2026
52 checks passed
Sign up for freeto 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

@emberb170d@ulises-jeremias