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

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?

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
@sonarqubecloud
Copy link

sonarqubecloud 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