Skip to content

feat(dev): add male_first_name,female_first_name, jp locale#1217

Merged
ST-DDT merged 4 commits intofaker-js:mainfrom
satoc0629:feat/add_jp_male_and_female_first_name
Aug 2, 2022
Merged

feat(dev): add male_first_name,female_first_name, jp locale#1217
ST-DDT merged 4 commits intofaker-js:mainfrom
satoc0629:feat/add_jp_male_and_female_first_name

Conversation

@satoc0629
Copy link
Copy Markdown
Contributor

No description provided.

@satoc0629 satoc0629 requested a review from a team as a code owner July 31, 2022 13:41
@xDivisionByZerox xDivisionByZerox added p: 1-normal Nothing urgent c: locale Permutes locale definitions m: person Something is referring to the person module labels Jul 31, 2022
@xDivisionByZerox xDivisionByZerox added this to the v7 - Current Major milestone Jul 31, 2022
@xDivisionByZerox xDivisionByZerox requested a review from a team July 31, 2022 13:45
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 31, 2022

Codecov Report

Merging #1217 (c4cbf6d) into main (5a397e0) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1217      +/-   ##
==========================================
+ Coverage   99.62%   99.63%   +0.01%     
==========================================
  Files        2153     2155       +2     
  Lines      236543   236831     +288     
  Branches      980      984       +4     
==========================================
+ Hits       235655   235967     +312     
+ Misses        867      843      -24     
  Partials       21       21              
Impacted Files Coverage Δ
src/locales/ja/name/female_first_name.ts 100.00% <100.00%> (ø)
src/locales/ja/name/index.ts 100.00% <100.00%> (ø)
src/locales/ja/name/male_first_name.ts 100.00% <100.00%> (ø)
src/modules/internet/user-agent.ts 88.09% <0.00%> (+6.34%) ⬆️

Copy link
Copy Markdown
Member

@ST-DDT ST-DDT left a comment

Choose a reason for hiding this comment

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

Reviewed using google translate. Looks good to me.

@ST-DDT ST-DDT requested a review from a team August 1, 2022 17:26
Copy link
Copy Markdown
Member

@ejcheng ejcheng left a comment

Choose a reason for hiding this comment

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

Also reviewed using Google Translate, lgtm

@ST-DDT ST-DDT enabled auto-merge (squash) August 2, 2022 08:33
@ST-DDT ST-DDT merged commit ad885ea into faker-js:main Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: locale Permutes locale definitions m: person Something is referring to the person module p: 1-normal Nothing urgent

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants