Skip to content

Remove minified version of script#189

Open
martey wants to merge 1 commit intomailcheck:masterfrom
mobolic:remove-minified-version
Open

Remove minified version of script#189
martey wants to merge 1 commit intomailcheck:masterfrom
mobolic:remove-minified-version

Conversation

@martey
Copy link
Copy Markdown

@martey martey commented May 6, 2022

Do not include minified version of script as part of source code and remove pre-commit hook. Because minified code can be easily generated from the unminified code, committing two codes of the script is an anti-pattern.

Do not include minified version of script as part of source code and
remove pre-commit hook.  Because minified code can be easily generated
from the unminified code, committing two codes of the script is an
anti-pattern.
@ferreiro
Copy link
Copy Markdown

ferreiro commented Oct 22, 2022

Hey @martey , this library is no longer maintained, and agreed that keeping the minified is a bad practise.

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.

2 participants