Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

feat: Japanese language support #922

Merged
merged 6 commits into from
Mar 6, 2020

Conversation

bart-jansen
Copy link
Member

Japanese translation verified by @aykhara and other native Japanese speakers

@codecov
Copy link

codecov bot commented Nov 7, 2019

Codecov Report

Merging #922 into develop will decrease coverage by 0.09%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           develop    #922     +/-   ##
=========================================
- Coverage     87.9%   87.8%   -0.1%     
=========================================
  Files          129     131      +2     
  Lines         4845    4963    +118     
  Branches       922     984     +62     
=========================================
+ Hits          4259    4358     +99     
- Misses         584     603     +19     
  Partials         2       2
Impacted Files Coverage Δ
src/common/strings.ts 100% <ø> (ø) ⬆️
src/common/localization/ja.ts 100% <100%> (ø)
src/providers/export/pascalVOC.ts 79.44% <0%> (-4.77%) ⬇️
src/providers/export/cntk.ts 96.96% <0%> (-0.65%) ⬇️
src/setupTests.js 100% <0%> (ø) ⬆️
src/providers/export/testAssetsSplitHelper.ts 100% <0%> (ø)
...nents/common/connectionPicker/connectionPicker.tsx 93.54% <0%> (+3.07%) ⬆️
src/common/ipcRendererProxy.ts 66.66% <0%> (+4.76%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 745e854...09f284e. Read the comment docs.

Copy link
Contributor

@PIC123 PIC123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Copy link
Contributor

@wbreza wbreza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for this contribution!

@peaceiris
Copy link

Whitespace like the following should be removed.

- ローカル プロジェクトを開く
+ ローカルプロジェクトを開く

@bart-jansen
Copy link
Member Author

bart-jansen commented Dec 12, 2019

Thanks a lot @peaceiris ! Really appreciate the feedback, got it verified by @aykhara and everything looks great and is added in bf0d273

@PIC123 @wbreza can you guys merge this?

@@ -112,7 +112,7 @@ export const japanese: IAppStrings = {
visitedAssets: "訪問済みアセット(${count})", // Visited Assets (${count}),
taggedAssets: "タグ付きアセット(${count})", // Tagged Assets (${count}),
nonTaggedAssets: "タグ付けされていないアセット(${count})", // Not Tagged Assets (${count}),
nonVisitedAssets: " 未訪問ないアセット(${count})", // Not Visited Assets (${count}),
nonVisitedAssets: "未訪問ないアセット(${count})", // Not Visited Assets (${count}),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I mentioned removing "ない"...

@mydiemho mydiemho closed this Mar 5, 2020
@mydiemho mydiemho reopened this Mar 5, 2020
@sonarcloud
Copy link

sonarcloud bot commented Mar 5, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@pjlittle pjlittle changed the base branch from master to develop March 6, 2020 14:50
@pjlittle pjlittle merged commit ed2d28f into microsoft:develop Mar 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants