-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
added github actions to check broken links #360
Conversation
I would just looking at one of the following Actions to provide the functionality. I kinda like https://github.com/lycheeverse/lychee-action because it's still active in development. A lot fo the others seemed to have just released one version and forgot about it. The one you selected, celinekurpershoek/link-checker, hasn't been touched since last October. So I would prefer an active community. |
Yeah makes sense, I'll update the pr |
@lakshya8066 Thank you |
Marking this as Hacktoberfest accepted since this is a variable update to our workflow and a great amount of research and testing that goes into it. |
Thank you @Stargator :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost ready, just drop the create issue logic and I think this is good.
.github/workflows/links.yml
Outdated
env: | ||
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}} | ||
|
||
- name: Create Issue From File |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need to create an issue as if the Workflow fails, we'd just have the PR branch updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay, I will update the PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to be sure, we don't need the environment variable too right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so.
@lakshya8066 It seems the Links workflow was not executed. Maybe we do need the secret token? |
Right, let's add it and see... |
Co-authored-by: Stargator <7527155+Stargator@users.noreply.github.com>
@lakshya8066 Yep, now the action is running. |
@lakshya8066 It's working, but there is an error for one link and yet the Action seems to be passed as Successful in the PR. I see it is returning exit code 2, is there someway to ensure it fails on error? |
@Stargator yes and I believe it stops checking once it gets a broken link or maybe it is just a coincidence. |
Updated to fail the test when a broken link is encountered Co-authored-by: Stargator <7527155+Stargator@users.noreply.github.com>
added checks for https
@lakshya8066 I made a few updates to resolve some errors. But now it's failing! YAY! 💯 Last steps now:
|
@Stargator They can be accessed at https://exercism.org/docs/tracks/dart |
@lakshya8066 I am approving this PR and merging it. But, I need you to understand that normally I would require all the checks to pass. That is why I expected the URLs to be fixed in this branch so the new link checker can verify everything is good. So do understand in the future for this repo or any future repo you contribute to. If there are fixes related to an existing PR it's probably best to do it all in one branch. Thanks again for all your work and patience, I really appreciate finally having this action in our workflow. 😄 🥇 |
@Stargator absolutely, I will keep that in mind. I realized the same after creating the other PR. |
Fixes #357