Skip to content

fix(locale): change id_ID name to Indonesian#1401

Merged
Shinigami92 merged 3 commits intofaker-js:mainfrom
fzn0x:patch-4
Sep 29, 2022
Merged

fix(locale): change id_ID name to Indonesian#1401
Shinigami92 merged 3 commits intofaker-js:mainfrom
fzn0x:patch-4

Conversation

@fzn0x
Copy link
Copy Markdown
Contributor

@fzn0x fzn0x commented Sep 27, 2022

The citizens of Indonesia should be called Indonesian

I notice that only Indonesian whos using their country name for calling the citizen that originally from Indonesia, it's should be Indonesian

@fzn0x fzn0x requested a review from a team September 27, 2022 11:45
@fzn0x fzn0x requested a review from a team as a code owner September 27, 2022 11:45
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 27, 2022

Codecov Report

Merging #1401 (cc27348) into main (7abeae5) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1401   +/-   ##
=======================================
  Coverage   99.62%   99.62%           
=======================================
  Files        2163     2163           
  Lines      241303   241303           
  Branches     1017     1017           
=======================================
  Hits       240396   240396           
  Misses        886      886           
  Partials       21       21           
Impacted Files Coverage Δ
src/locales/id_ID/index.ts 100.00% <100.00%> (ø)

@fzn0x fzn0x changed the title fix: The citizens of Indonesia should be called Indonesian fix(locale): The citizens of Indonesia should be called Indonesian Sep 27, 2022
@ejcheng ejcheng added p: 1-normal Nothing urgent c: locale Permutes locale definitions labels Sep 27, 2022
@ejcheng ejcheng added this to the v7 - Current Major milestone Sep 27, 2022
@ejcheng ejcheng requested a review from a team September 27, 2022 12:47
@ST-DDT ST-DDT added the s: accepted Accepted feature / Confirmed bug label Sep 27, 2022
@pkuczynski
Copy link
Copy Markdown
Member

Change seems to be correct, but worth to notice, that this is not about citizens of Indonesia, but about locale name. I will update the PR title.

@pkuczynski pkuczynski changed the title fix(locale): The citizens of Indonesia should be called Indonesian fix(locale): rename id_ID locale to Indonesian Sep 27, 2022
@pkuczynski pkuczynski changed the title fix(locale): rename id_ID locale to Indonesian fix(locale): rename id_ID locale to Indonesian Sep 27, 2022
@pkuczynski pkuczynski changed the title fix(locale): rename id_ID locale to Indonesian fix(id_ID): change locale name to Indonesian Sep 27, 2022
@fzn0x
Copy link
Copy Markdown
Contributor Author

fzn0x commented Sep 28, 2022

Thanks @pkuczynski !

@Shinigami92 Shinigami92 changed the title fix(id_ID): change locale name to Indonesian fix(locale): change id_ID name to Indonesian Sep 29, 2022
@Shinigami92 Shinigami92 merged commit ec0181e into faker-js:main Sep 29, 2022
@fzn0x fzn0x deleted the patch-4 branch September 29, 2022 17:25
wael-fadlallah pushed a commit to wael-fadlallah/faker that referenced this pull request Oct 14, 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 p: 1-normal Nothing urgent s: accepted Accepted feature / Confirmed bug

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants