markdown-it/linkify-it

Add .app TLD to tlds_default

buu700 opened this issue · 6 comments

buu700 commented

The comment says not to submit a PR, so I won't do that, but .app is a pretty common/standard TLD nowadays and so seems to make sense in the default list.

puzrin commented

I see no reason for breaking changes, while you can configure TLD list as you wish.

ursnj commented

Yeah this is very much needed in recent days.

  • 👍1
buu700 commented

Just to clarify, I do have .app configured in my own application; I'm just suggesting that it would make more sense to have as a default.

puzrin commented

Any change of defaults is breaking. We avoid breaking changes when other alternatives available. Let's keep things as is. Sorry for inconvenience.

buu700 commented

Got it, in that case it may just be something to consider for version 5.0.0.

puzrin commented

v5 may happen next day after "never" :). At current moment i have no ideas for v5.

  • 👍1