Skip to content

"net" listed in defaultTopLevelDomains twice#159

Open
stereoscott wants to merge 1 commit intomailcheck:masterfrom
stereoscott:patch-1
Open

"net" listed in defaultTopLevelDomains twice#159
stereoscott wants to merge 1 commit intomailcheck:masterfrom
stereoscott:patch-1

Conversation

@stereoscott
Copy link
Copy Markdown

"net" is already listed on line 32, but is repeated again on line 34 between "in" and "net.au".

martey added a commit to mobolic/mailcheck that referenced this pull request May 6, 2022
* stereoscott/patch-1:
  "net" listed in defaultTopLevelDomains twice
@ferreiro
Copy link
Copy Markdown

Hey @stereoscott, good catch on the duplication! this library is no longer maintained unfortunately :/

I rewrote it entirely and update it for 2022 and onward: https://github.com/ZooTools/email-spell-checker

  • 💙 Written in TypeScript and removed jQuery
  • ✅ Fixed a couple of bugs like this or this
  • 🚀 Reduced bundle size to <2KB.

  • ✨ Update TLDs (69+) and added modern startup domains (like .io, .so, .xyz or .dev)
  • 🙏 Implemented suggestions that people made in this repo that were ignored.

Link: https://github.com/ZooTools/email-spell-checker

Come check it out and give it a ⭐️ for the effort.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants