Closed karlhorky closed 1 week ago
Thanks for sharing @karlhorky! Similar to https://github.com/remarkjs/remark-lint-no-dead-urls/issues/50 this feels more like a reporter than something that needs-to-be/should be fixed here.
It is intentional that markdown syntax is used in message reasons.
If GitHub Actions has a “linkifier” that doesn’t work, report it with them.
If GitHub Actions has a “linkifier” that doesn’t work, report it with them.
I would, but the likelihood of GitHub fixing bugs is... unlikely. There are many open bugs since years that are very publicly reported. After many 10s or 100s of hours of reporting bugs without any feedback, it's becoming clear that it's not worth reporting bugs to GitHub.
In my opinion, projects should instead work around the bugs in GitHub. But I don't think this opinion is shared by the MDX team.
I guess I'll have to come up with a way to fix the output of remark-lint-no-dead-urls
using sed
or something (or maybe it could be a Refined GitHub feature). The suggestion to build a custom reporter sounds like a lot of work + research + investment.
markdown syntax is used in message reasons
I can understand that. However, I guess I would argue that using a inline code block with backticks around a URL is unnecessary and error-prone. I usually try to avoid having any characters close to URLs that I write in any format, because of linkification problems like this, across lots of software, not only GitHub.
In my opinion, projects should instead work around the bugs in GitHub. But I don't think this opinion is shared by the MDX team.
You can make a GH reporter.
However, I guess I would argue that using a inline code block with backticks around a URL is unnecessary and error-prone
I disagree. URLs are not grammatical values. They are code.
What about a middle path - adding a space inside the backticks on both ends? Then the URL still stays as code, but the backtick character isn't directly next to the URL.
I don’t really mind whether or not the URL is surrounded by backticks, but I agree with Titus that this is a problem with GitHub’s link detection.
I do not think that is a middle path; I think that is ugly for everybody else. If GH doesn’t work well for you, make a reporter to make it work well, or use a different tool that works well, such as your local editor.
Ok, I understand this will not be accepted, but one last thing:
If GH doesn't work well for you
This is actually not as much about me (I'll probably remember, for a few weeks / months at least). And I'll probably build some workaround, so that I and others on our team don't need to remember.
But it's more about confusing all users who click on remark-lint-no-dead-urls
links in a very common tool / environment. (who would not have the workaround installed, so thus have broken links by default)
But it's more about confusing all users who click on remark-lint-no-dead-urls links in a very common tool / environment. (who would not have the workaround installed, so thus have broken links by default)
I hear you, understand you, and appreciate your viewpoint. There can be (and are) bugs in common and popular tools that should be fixed right there at the origin. Many are quickly fixed. 🎉
If any get stuck (case and point https://github.com/jestjs/jest/issues/9430 still blocks usage of ESM) the recommendation for users who are frustrated is to move to another tool entirely.
Mandatory XKCD reference 😉
Initial checklist
Affected packages and versions
remark-lint-no-dead-urls@2.0.0
Link to runnable example
No response
Steps to reproduce
remark-lint-no-dead-urls
on GitHub Actions and click on a failing link in the output%60
, likely leading to confusionYou can see it here, the backtick is also underlined and has a gray color:
As text:
Expected behavior
Links do not include extra suffix or prefix characters which will be linkified by GitHub
Actual behavior
Links include the extra suffix backtick character which will be linkified by GitHub
Runtime
No response
Package manager
No response
OS
No response
Build and bundle tools
No response