Closed steppi closed 9 months ago
Name | Link |
---|---|
Latest commit | 1309dd80b150c4ca9c65a96749df213492fe9f25 |
Latest deploy log | https://app.netlify.com/sites/numpy-org/deploys/657b42498dbdec00081b783c |
Deploy Preview | https://deploy-preview-698--numpy-org.netlify.app/ja/news |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Thanks @jarrodmillman for pointing out the merge artifacts, @AtsushiSakai for the review, and @tkoyama010 for the suggestions. I think this is ready now if someone wants to merge it.
Brief description of what is fixed or changed
Update Japanese translations. I saw on Crowdin that @AtsushiSakai translated the updated news content and adjusted some translations so making a PR to get these updates in.
Crowdin is completely out of sync with github because we didn't merge Crowdin's autogenerated PR, instead taking only the commits we wanted. In addition, translations on main also had some updates made directly without using Crowdin. My process here is to do a scripted rebase on the commit where l10n_main diverges from main which filters out all commits not from the language of interest, and squashes together commits that touch the same file (fortunately Crowdin doesn't make commits that touch multiple files). I then rebase this on main and manually clean up the merge conflicts.
@AtsushiSakai, please let me know if everything looks Ok. cc @tkoyama010 in case you want to take a look too.