-
-
Notifications
You must be signed in to change notification settings - Fork 374
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
Upgrade autolinker #356
Upgrade autolinker #356
Conversation
Refs #340 /~https://github.com/gregjacobs/Autolinker.js/releases - fixed test - dropped twitter option - dropped rollup commonjs plugin as autolinker is distributed as esm now
3 similar comments
Friendly ping @jonschlinkert @doowb @shockey |
@TrySound Are you sure you fixed and didn't break the test? See gregjacobs/Autolinker.js#278 |
Damn, missed your comment in that PR |
I see in this test that such behaviour is expected. |
I extended version range so users could upgrade autolinker when the new version is out. If the behaviour will not fit to users they may copy paste lunkify plugin into own code base and use any lib and version without any loss. Linkify plugin is a separate entry point now. |
Refs #340 and /~https://github.com/gregjacobs/Autolinker.js/releases
/cc @kibertoad