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.

Sign up for GitHub

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

Jump to bottom

feat(locale): Add finance module for zh_CN #2370

Merged
merged 11 commits into from Sep 17, 2023
Merged

feat(locale): Add finance module for zh_CN #2370

merged 11 commits into from Sep 17, 2023

Conversation

CoolPlayLin
Copy link
Contributor

No description provided.

@CoolPlayLin CoolPlayLin requested a review from a team as a code owner September 1, 2023 14:52
@codecov
Copy link

codecov bot commented Sep 1, 2023 •
edited

Codecov Report

Merging #2370 (b25920c) into next (a7d25fa) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files 
@@            Coverage Diff             @@
##             next    #2370      +/-   ##
==========================================
- Coverage   99.61%   99.60%   -0.01%     
==========================================
  Files        2793     2800       +7     
  Lines      251586   252410     +824     
  Branches     1100     1094       -6     
==========================================
+ Hits       250610   251421     +811     
- Misses        949      962      +13     
  Partials       27       27
Files Changed Coverage Δ
src/locales/zh_CN/finance/account_type.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/finance/credit_card/index.ts 100.00% <100.00%> (ø)
...rc/locales/zh_CN/finance/credit_card/mastercard.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/finance/credit_card/visa.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/finance/currency.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/finance/index.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/finance/transaction_type.ts 100.00% <100.00%> (ø)
src/locales/zh_CN/index.ts 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@ST-DDT ST-DDT added p: 1-normal Nothing urgent c: locale Permutes locale definitions m: finance Something is referring to the finance module labels Sep 1, 2023
ST-DDT
ST-DDT previously approved these changes Sep 1, 2023
View reviewed changes
@ST-DDT ST-DDT requested review from a team September 1, 2023 21:04
import-brain
import-brain previously approved these changes Sep 4, 2023
View reviewed changes
@import-brain import-brain added the needs rebase There is a merge conflict label Sep 4, 2023
@import-brain
Copy link
Member

Hi, please rebase this pull request.

@CoolPlayLin CoolPlayLin dismissed stale reviews from import-brain and ST-DDT via 278721b September 8, 2023 08:59
@CoolPlayLin CoolPlayLin removed their assignment Sep 8, 2023
@CoolPlayLin
Copy link
Contributor Author

done

@ST-DDT ST-DDT removed the needs rebase There is a merge conflict label Sep 8, 2023
ST-DDT
ST-DDT previously approved these changes Sep 8, 2023
View reviewed changes
@ST-DDT ST-DDT requested review from a team September 8, 2023 09:29
import-brain
import-brain previously approved these changes Sep 9, 2023
View reviewed changes
@CoolPlayLin CoolPlayLin mentioned this pull request Sep 16, 2023
Merged
@ST-DDT ST-DDT enabled auto-merge (squash) September 17, 2023 08:45
@ST-DDT ST-DDT enabled auto-merge (squash) September 17, 2023 08:46
@ST-DDT ST-DDT merged commit aea4c9b into faker-js:next Sep 17, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthewmayer matthewmayer matthewmayer approved these changes

@ST-DDT ST-DDT ST-DDT approved these changes

@import-brain import-brain Awaiting requested review from import-brain

Assignees
No one assigned
Labels
c: locale Permutes locale definitions m: finance Something is referring to the finance module p: 1-normal Nothing urgent
Projects
None yet
Milestone
vAnytime
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@CoolPlayLin @import-brain @matthewmayer @ST-DDT @Shinigami92