README in English, like the rest of the ecosystem - #3
Merged
Conversation
The docs site was already in English; the README was not, and a package published on Packagist is read by people who do not speak Italian. The README is the first thing they see, so it is the last place to keep a second language. Translated in full, keeping the register rather than the words: every claim still says which failure it prevents, which is the reason those sentences are written the way they are. Structure is untouched — same 29 headings, same tables row for row, same code blocks, same links (the only change is the anchor #test → #tests, following its heading). The one Italian comment left in the doc site (assets/custom.css) goes to English too. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EZ4u82zdoa8naYm75kppMC
Deploying laravel-routines with
|
| Latest commit: |
88e0354
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://a53cad13.laravel-routines.pages.dev |
| Branch Preview URL: | https://task-english-readme.laravel-routines.pages.dev |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The docs site was already entirely in English — the README was not. A package published on Packagist is read by people who do not speak Italian, and the README is the first thing they see, so it is the last place to keep a second language.
What changed
The README, translated in full — keeping the register rather than the words. Every claim still says which failure it prevents: that is the reason those sentences are written the way they are, and a flatter translation would have kept the claims while losing the arguments.
The one Italian comment left in the doc site (
docs-site/assets/custom.css) goes to English too.Structure is untouched
Verified mechanically, not by eye:
¹ The single difference is the internal anchor
#test→#tests, following its renamed heading — checked to resolve.Both images still resolve, and
npm run buildindocs-site/still generates 8 pages with zero warnings.Not in scope, and deliberately so
CLAUDE.mdstays in Italian: it is guidance for the team and for AI agents working in the repo, not community-facing, and it matches the convention inlaravel-iam-agentsand the other packages.laravel-iam-agentsandlaravel-floware Italian insrc/too). Changing them is a large, separate diff — say the word if you want it.Generated by Claude Code