Skip to content

docs: add validation examples for all 5 recently added South Asian countries (ID, BD, PK, LK, NP) - #20

Merged
ankitgadling merged 3 commits into
ankitgadling:mainfrom
harshadarayate:main
Oct 7, 2025
Merged

docs: add validation examples for all 5 recently added South Asian countries (ID, BD, PK, LK, NP)#20
ankitgadling merged 3 commits into
ankitgadling:mainfrom
harshadarayate:main

Conversation

@harshadarayate

Copy link
Copy Markdown
Contributor

📌 Pull Request

Description

Updated README.md with usage examples for the five recently added countries from [commit #16]: Indonesia (ID), Bangladesh (BD), Pakistan (PK), Sri Lanka (LK), and Nepal (NP).

Addresses the issue by adding:

  • Code snippets showing validate() calls (with country codes and full names).
  • Highlighted expected outputs (**Expected: True**).

Changes Made

  • Added/Updated postal regex for a new country
  • Improved documentation

Screenshots

image image

…countries (ID, BD, PK, LK, NP)

- Includes code snippets for `validate()` calls with sample postal codes from commit ankitgadling#16
- Highlights expected `True` outputs in bold for clarity
- Consistent with existing examples (IN, US); tested locally via `pip install -e .`
…e_file

docs: added validation examples for all 5 recently added South Asian …
@harshadarayate

Copy link
Copy Markdown
Contributor Author

heyy @ankitgadling ! Could ypu please review and merge the updated work!

@ankitgadling

ankitgadling commented Oct 7, 2025

Copy link
Copy Markdown
Owner

heyy @ankitgadling ! Could ypu please review and merge the updated work!

The changes look good could you help improve the overall structure and readability of the README? For example, making sure sections are well organized, improving clarity, or adding missing details where needed

Improved the overall structure and readability of the README file.
@harshadarayate

harshadarayate commented Oct 7, 2025

Copy link
Copy Markdown
Contributor Author

heyy @ankitgadling ! Improved the overall structure and readability of the README file.

@ankitgadling
ankitgadling merged commit b6d790d into ankitgadling:main Oct 7, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants