Skip to content
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

Translation manager update #589

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Conversation

wbazant
Copy link
Collaborator

@wbazant wbazant commented Nov 14, 2024

Part of #538.

Fyi @mpastoring I've changed our script a bit while writing up instructions for translation issues. If you have ideas for the process or would be up for translating some of the pages, please feel free!

--migrate always migrates how we want it
--list-in-component just lists
@wbazant
Copy link
Collaborator Author

wbazant commented Nov 14, 2024

These were the AI instructions to change the script:

history | grep aider | grep trans
  627  ,,aider scripts/translation_manager.py -m "Fix this error that happens for parser.print_help(): ValueError: unsupported format character '{' (0x7b) at index 9"
  630  ,,aider scripts/translation_manager.py -m "Add option --list-in-component that just needs component path, and lists all the keys in the file"
  633  ,,aider scripts/translation_manager.py -m "When doing --list-in-component, cope with json path not being provided " # that just needs component path, and lists all the keys in the file"
  650  ,,aider scripts/translation_manager.py -m "Do not create any new locale files when doing --migrate"
  654  ,,aider scripts/translation_manager.py -m "When logging for --migrate, print the yaml and json file with -> in between"
  656  ,,aider scripts/translation_manager.py -m "When logging for --migrate and file doesn't exist, print the expected path"
  662  ,,aider scripts/translation_manager.py -m "For --migrate, count added and edited keys and log them"
  671  ,,aider scripts/translation_manager.py -m "Remove --remove-params and --replace-placeholders as separate options. Instead, always do these operations for the migrated keys when migrating"

@wbazant wbazant merged commit 2e9e59a into falling-fruit:main Nov 14, 2024
1 check passed
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.

1 participant