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.

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

refactor(locale)!: change en_IND to en_IN #1448

Merged
merged 4 commits into from
Oct 14, 2022

Conversation

wael-fadlallah
Copy link
Contributor

@wael-fadlallah wael-fadlallah commented Oct 14, 2022

For #1354

@wael-fadlallah wael-fadlallah requested a review from a team October 14, 2022 21:18
@wael-fadlallah wael-fadlallah requested a review from a team as a code owner October 14, 2022 21:18
@import-brain import-brain added p: 1-normal Nothing urgent c: locale Permutes locale definitions breaking change Cannot be merged when next version is not a major release needs rebase There is a merge conflict labels Oct 14, 2022
@Shinigami92
Copy link
Member

@wael-fadlallah looks like you didn't changed first to the next / default branch

@codecov
Copy link

codecov bot commented Oct 14, 2022

Codecov Report

Merging #1448 (4173eef) into next (b99ff71) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #1448      +/-   ##
==========================================
- Coverage   99.62%   99.61%   -0.02%     
==========================================
  Files        2164     2164              
  Lines      236810   236810              
  Branches     1003      999       -4     
==========================================
- Hits       235911   235887      -24     
- Misses        877      901      +24     
  Partials       22       22              
Impacted Files Coverage Δ
src/locales/en_IN/address/city.ts 100.00% <ø> (ø)
src/locales/en_IN/address/city_name.ts 100.00% <ø> (ø)
src/locales/en_IN/address/default_country.ts 100.00% <ø> (ø)
src/locales/en_IN/address/index.ts 100.00% <ø> (ø)
src/locales/en_IN/address/postcode.ts 100.00% <ø> (ø)
src/locales/en_IN/address/state.ts 100.00% <ø> (ø)
src/locales/en_IN/address/state_abbr.ts 100.00% <ø> (ø)
src/locales/en_IN/address/street.ts 100.00% <ø> (ø)
src/locales/en_IN/company/index.ts 100.00% <ø> (ø)
src/locales/en_IN/company/suffix.ts 100.00% <ø> (ø)
... and 12 more

@Shinigami92 Shinigami92 removed the needs rebase There is a merge conflict label Oct 14, 2022
@Shinigami92 Shinigami92 requested review from a team October 14, 2022 22:13
@Shinigami92 Shinigami92 linked an issue Oct 14, 2022 that may be closed by this pull request
10 tasks
@Shinigami92 Shinigami92 changed the title refactor(locale)!: changed en_IND locale to en_IN refactor(locale)!: changed en_IND to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 enabled auto-merge (squash) October 14, 2022 22:14
@Shinigami92 Shinigami92 changed the title refactor(locale)!: changed en_IND to en_IN refactor(locale)!: change en_IND to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 changed the title refactor(locale)!: change en_IND to en_IN refactor(locale)!: changed the en_IND locale to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 changed the title refactor(locale)!: changed the en_IND locale to en_IN refactor(locale)!: change the en_IND locale to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 enabled auto-merge (squash) October 14, 2022 23:35
@ST-DDT ST-DDT added the s: accepted Accepted feature / Confirmed bug label Oct 14, 2022
@Shinigami92 Shinigami92 changed the title refactor(locale)!: change the en_IND locale to en_IN refactor(locale)!: changed the en_IND locale to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 changed the title refactor(locale)!: changed the en_IND locale to en_IN refactor(locale)!: change en_IND to en_IN Oct 14, 2022
@Shinigami92 Shinigami92 enabled auto-merge (squash) October 14, 2022 23:39
@Shinigami92 Shinigami92 merged commit a7cd422 into faker-js:next Oct 14, 2022
@matthewmayer
Copy link
Contributor

matthewmayer commented Oct 15, 2022

Can en_IND be made an alias (throwing a warning) for en_IN for v8 and then removed completely in v9?

@Shinigami92
Copy link
Member

Can en_IND be made an alias (throwing a warning) for en_IN for v8 and then removed completely in v9?

@matthewmayer Please open a PR to tackle this, or open a new issue so we can make that visible and not miss it before releasing v8.0.
A comment in a already closed/merged PR will likely go under and unseen ⚠️

@matthewmayer
Copy link
Contributor

Can en_IND be made an alias (throwing a warning) for en_IN for v8 and then removed completely in v9?

@matthewmayer Please open a PR to tackle this, or open a new issue so we can make that visible and not miss it before releasing v8.0. A comment in a already closed/merged PR will likely go under and unseen ⚠️

sure #1454

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Cannot be merged when next version is not a major release 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.

en_IND locale should be en_IN
5 participants