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

Ability to run pre-translate for whole project #830

Closed
whitipet opened this issue Aug 2, 2024 · 2 comments · Fixed by #845
Closed

Ability to run pre-translate for whole project #830

whitipet opened this issue Aug 2, 2024 · 2 comments · Fixed by #845

Comments

@whitipet
Copy link

whitipet commented Aug 2, 2024

When I call pre-translate, I need to specify either --file --directory. I have not found a way to specify all files at once, or the root folder.
crowdin pre-translate --method tm --translate-with-perfect-match-only --translate-untranslated-only
✔️ Fetching project info
❌ The '--file' or '--directory' parameter is required for this type of project

A possible solution is to add * for --directory, or all for --file as possible values

@andrii-bodnar
Copy link
Member

andrii-bodnar commented Aug 7, 2024

Thanks for the suggestion!

I think we can add all as a possible value for the --file parameter. It should also consider the --branch parameter and pre-translate all the files from the specified branch only.

It shouldn't be a default value but should be specified explicitly.


UPD: We've decided to simply allow the pre-translate command to be run without the --file or --directory parameters. In this case, it will pre-translate the whole project by default.

@whitipet
Copy link
Author

whitipet commented Aug 7, 2024

Great, that will help a lot

@yevheniyJ yevheniyJ self-assigned this Sep 21, 2024
@yevheniyJ yevheniyJ linked a pull request Sep 21, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants