-
Notifications
You must be signed in to change notification settings - Fork 424
docs: LLM translation. #2113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
docs: LLM translation. #2113
Conversation
|
Does GitHub not show the changed workflow file? It just shows the massive list of docs instead. |
|
No workflow yet as I'm not sure when it should run. Just ran the |
|
Ah, I closed the docs folder and didn't see the file. Thanks, will take a look. |
|
We'll need to add integration with super linter so that AI can automatically fix what it breaks. |
|
Can we simply exclude the Edit: probably a question for @dunglas |
|
I added a |
|
As it's manually triggered, we can't trigger it before it's merged into the default branch (main) iirc. push:
branches:
- main
paths:
- 'docs/*'Should be this though, so it's automatically opened after docs change. |
|
Makes sense, I adjusted the workflow to only trigger on changed english *.md files. Still not sure how to test it manually. I can also remove the docs changes themselves to make this PR slimmer. |
|
Yeah should remove the doc changes. I think we can experiment with it more once the workflow file is merged into main. |
| uses: shivammathur/setup-php@v2 | ||
| with: | ||
| php-version: '8.5' | ||
| - name: run translation script |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe could we use gemini-cli or copilot-cli directly, and make it running the linter and fixing the issues?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The script doesn't need to be PHP, can also be bash. You'll usually get more consistent results if input and output of the LLM are minimized though. Prompting the cli directly makes this too unpredictable and noisy IMO, but we can try as an experiment if you want.
As mentioned in #2108, this PR adds a script to translate all English docs via Gemini. LLM translations are generally pretty good, I can't attest to the quality of these translations though as I don't speak any of the languages. wdyt?