awesome-lint icon indicating copy to clipboard operation
awesome-lint copied to clipboard

Getting Invalid URL remark-lint:double-link, false positive?

Open probonopd opened this issue 4 years ago • 8 comments

Hello @sindresorhus I am getting a validation error which I suspect to be a false positive:

My list:

  • Source: https://raw.githubusercontent.com/AppImage/awesome-appimage/e9dcc94fab9e43bb213083318a61a6964e6a0fc0/README.md
  • Rendered: https://github.com/AppImage/awesome-appimage/blob/e9dcc94fab9e43bb213083318a61a6964e6a0fc0/README.md

The validation error:

1:1  Invalid URL: http:#appimage-discovery        remark-lint:double-link

Full log:

https://github.com/AppImage/awesome-appimage/runs/939913779?check_suite_focus=true#step:5:8

What does it mean, how can I fix it? Thank you very much.

probonopd avatar Aug 03 '20 09:08 probonopd

// @Scrum

sindresorhus avatar Aug 04 '20 22:08 sindresorhus

@probonopd @sindresorhus Thanks for the find, I have already made the necessary changes to the package remark-lint-double-link, let me know if something goes wrong after the update.

Scrum avatar Aug 05 '20 07:08 Scrum

Thank you very much @Scrum - this solves the particular issue.

But now I am getting what looks like other false positives:

  ✖  193:3   https://www.youtube.com/watch?v=XTGn_JqmDu0                                    remark-lint:double-link
  ✖  203:3   https://www.youtube.com/watch?v=nzZ6Ikc7juw                                    remark-lint:double-link
  ✖  204:3   https://www.youtube.com/watch?v=7fPShv-8Z_4                                    remark-lint:double-link

Could it be that what follows after the ? is ignored?

  • Source: https://raw.githubusercontent.com/AppImage/awesome-appimage/9932be888d11967417bd3ba067cd16de219ad665/README.md
  • Full log: https://github.com/AppImage/awesome-appimage/runs/975575451

probonopd avatar Aug 12 '20 11:08 probonopd

@probonopd Hi, it will take me a while to explore the possibility of including parameters in the plugin

Scrum avatar Aug 12 '20 12:08 Scrum

@Scrum until then, is there an ignore for this particular rule but only applying to a specific URL? IE: ignore all YouTube links?

jthegedus avatar Aug 12 '20 22:08 jthegedus

@probonopd @jthegedus

@Scrum until then, is there an ignore for this particular rule but only applying to a specific URL? IE: ignore all YouTube links?

Yes, now you can set the ignore flag for such link types, for example

- [puper-link-1](http://link-1)
- [puper-link-1](http://link-2)
<!--lint ignore double-link-->
- [Produce an AppImage that bundles everything with go-appimage](https://www.youtube.com/watch?v=XTGn_JqmDu0)
- [puper-link-1](http://link-1)
- [AppImage: Portable applications for Linux](https://www.youtube.com/watch?v=nzZ6Ikc7juw)
- [Comparing Linux Package Formats - Deb, Flatpak, AppImage, etc.](https://www.youtube.com/watch?v=7fPShv-8Z_4)

In this case, you will receive only one error '2:3-2:32: http://link-1',

Scrum avatar Aug 13 '20 06:08 Scrum

Also happening to me as well , not only YouTube either. Google scholar also results the same.

ugurkanates avatar Sep 27 '20 16:09 ugurkanates

resolved in https://github.com/Scrum/remark-lint-double-link/pull/6 Now query parameters will participate in validation

Scrum avatar Sep 28 '20 07:09 Scrum