Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add space after url in error message #32

Merged
merged 1 commit into from
May 5, 2024

Conversation

hugobuddel
Copy link
Contributor

Closes #31

Github will parse the trailing comma as part of the url when the warning appears in a github action log, so the link cannot be followed.

Arguably this is a github bug, and not a linkspector bug, because github does not include the comma in the URL in other places, like in issues.

Nevertheless, linkspector will probably be running in github actions quite often, so it would be good if the links are clickable from there without having to manually remove the comma.

Closes UmbrellaDocs#31

Github will parse the trailing comma as part of the url when the warning appears in a github action log, so the link cannot be followed.

Arguably this is a github bug, and not a linkspector bug, because github does not include the comma in the URL in other places, like in issues.

Nevertheless, linkspector will probably be running in github actions quite often, so it would be good if the links are clickable from there without having to manually remove the comma.
@gaurav-nelson gaurav-nelson changed the base branch from main to dev-0.3.0 May 5, 2024 06:29
@gaurav-nelson gaurav-nelson merged commit f10b735 into UmbrellaDocs:dev-0.3.0 May 5, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Add space before , after broken UNL
2 participants