Skip to content

feat(isLat, isLong) #1830: implemented isLat and iLong #1863

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

Closed

Conversation

fkrafi
Copy link

@fkrafi fkrafi commented Nov 9, 2021

isLat(str [, options]) | check if the string is a valid latitude coordinate in the format lat.

options is an object that defaults to { checkDMS: false }. Pass checkDMS as true to validate DMS(degrees, minutes, and seconds) latitude format.

isLong(str [, options]) | check if the string is a valid longitude coordinate in the format long.

options is an object that defaults to { checkDMS: false }. Pass checkDMS as true to validate DMS(degrees, minutes, and seconds) longitude format.

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)
  • Tests written (where applicable)

@codecov
Copy link

codecov bot commented Nov 9, 2021

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (47ee5ad) to head (50f2eb0).
Report is 263 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #1863   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          102       104    +2     
  Lines         2059      2072   +13     
  Branches       464       464           
=========================================
+ Hits          2059      2072   +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fkrafi
Copy link
Author

fkrafi commented Nov 9, 2021

@fncolon can you please accept this PR #1830

@fkrafi
Copy link
Author

fkrafi commented Nov 10, 2021

@profnandaa, @tux-tn, @ezkemboi, can you please have a look

@tux-tn
Copy link
Member

tux-tn commented Nov 15, 2021

Thank you for your PR, i'm wondering if this new validator/feature is needed since we already have isLatLong

@WikiRik
Copy link
Member

WikiRik commented Mar 28, 2025

In #1830 we decided that we will not accept a new validator for this, but we can add options to isLatLong to only validate one of the two

@WikiRik WikiRik closed this Mar 28, 2025
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.

3 participants