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

feat(location): add ne postcodes #2148

Merged
merged 4 commits into from
Jun 16, 2023
Merged

Conversation

matthewmayer
Copy link
Contributor

Nepali postcodes are 5 digits starting with [1-5]

https://www.gpo.gov.np/Home/Postalcode

Nepali postcodes are 5 digits starting with [1-5]
@matthewmayer matthewmayer requested a review from a team as a code owner May 11, 2023 04:22
@matthewmayer matthewmayer self-assigned this May 11, 2023
@matthewmayer matthewmayer added c: locale Permutes locale definitions m: location Something is referring to the location module p: 1-normal Nothing urgent labels May 11, 2023
@codecov
Copy link

codecov bot commented May 11, 2023

Codecov Report

Merging #2148 (d9333cc) into next (c3b540f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #2148   +/-   ##
=======================================
  Coverage   99.60%   99.60%           
=======================================
  Files        2620     2621    +1     
  Lines      245472   245476    +4     
  Branches     1154     1158    +4     
=======================================
+ Hits       244493   244515   +22     
+ Misses        952      934   -18     
  Partials       27       27           
Impacted Files Coverage Δ
src/locales/ne/location/index.ts 100.00% <100.00%> (ø)
src/locales/ne/location/postcode.ts 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@ST-DDT ST-DDT requested review from a team May 11, 2023 07:16
@ST-DDT ST-DDT added the s: accepted Accepted feature / Confirmed bug label May 11, 2023
Copy link
Member

@xDivisionByZerox xDivisionByZerox left a comment

Choose a reason for hiding this comment

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

I wasn't really able to figure out the zip codes from the source you provided.
https://www.nepalitrends.com/list-of-postal-codes-of-nepal/ did help me tho. Additionally, it has a full table of all possible postcodes, as the current templates could generate some invalid zip codes. But it's good enough IMO.

@ST-DDT ST-DDT enabled auto-merge (squash) June 16, 2023 20:32
@ST-DDT ST-DDT merged commit 36d1d3a into faker-js:next Jun 16, 2023
20 checks passed
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: location Something is referring to the location module 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.

None yet

4 participants