-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Website deploy is broken #8707
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
Comments
I suspect the generated i18n json file is invalid. Will need more investigation |
Oops. I almost forgot about this. I am unable to debug this without jest's crowdin api key. |
I don't have full access to that key. I don't have access to the Jest Crowdin settings either. @ericnakagawa do you happen to have access to Jest's Crowdin settings? @SimenB Do you know anyone else who might? |
I don't - I'd have suggested @ericnakagawa |
I have the key, endiliey I'll dm you it. |
So i was able to simulate the circleci environment with jest's crowdin api key but was not able to reproduce the error. The spanish translation is fine It seems to be a transient issue as website deploy has been working for jest now https://circleci.com/gh/facebook/jest/67767?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link From the error defined in the circleci, it also suggests that it is an error in crowdin-cli during download step defined here https://github.com/facebook/jest/blob/f857d309646154f6ae587ec744195453030d1f7e/.circleci/website.sh#L18-L24 The error itself is in fact a well known Java error (and docusaurus doesn't use java, only the crowdin-cli) https://stackoverflow.com/questions/22062056/a-jsonobject-text-must-begin-with-at-1-character-2-line-1-with-error Error: A JSONObject text must begin with '{' at 1 [character 2 line 1] Given that jest has so many translation files, it is more than possible that the files got corrupted during download, or some transient issue My recommendationChecking at https://crowdin.com/project/jest |
Seems fair to disable the 0% ones. If somebody wants to translate them, how would it be enabled again? |
It's unfortunate that crowdin-cli download is abit inflexible. I wish it can ignore downloading low % translated languages or download certain languages only For example, there are 39 languages defined in https://crowdin.com/project/jest and we have no choice but to download every single languages. Crowdin-cli download only has two choices download 1 language only crowdin-cli download -l <language_code> or download all enabled languages in crowdin crowdin-cli download A quick win-win solution but abit ugly for us all is to change to yarn crowdin-download -l es-ES
yarn crowdin-download -l uk
# other wanted languages No need to disable anything in crowdin. i can send a pr |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
See https://circleci.com/gh/facebook/jest/67119
Seems to explode when getting Spanish translations?
Error: A JSONObject text must begin with '{' at 1 [character 2 line 1]
@yangshun @endiliey @JoelMarcey (sorry... dunno who else to ask) is this something you know how to debug?
The text was updated successfully, but these errors were encountered: