-
Notifications
You must be signed in to change notification settings - Fork 17
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
Run ended on an exception #11
Comments
Are you running from version v1.0.1? I messed up some tags yesterday 😅 I'm now running the action but it can take a while since its testing the readme page with recursive urls. https://github.com/celinekurpershoek/link-checker/runs/1076144518?check_suite_focus=true#step:2:8 |
I've got v1.0.1, yes |
https://github.com/jasikpark/jasik-xyz/blob/master/.github/workflows/broken-link-check.yml name: Broken Link Check
on:
schedule:
- cron: 0 0 1 * * # run monthly
repository_dispatch: # run manually
types: [check-link]
jobs:
broken_link_checker_job:
runs-on: ubuntu-latest
name: Check for broken links
steps:
- name: Check for broken links
id: link-report
uses: celinekurpershoek/[email protected]
with:
# Required:
url: "https://jasik.xyz"
# optional:
honorRobotExclusions: false
ignorePatterns: "github,google"
recursiveLinks: true # Check all urls on all reachable pages (could take a while)
- name: Get the result
run: echo "${{steps.link-report.outputs.result}}" |
going to look into it ;) |
It looks like its having trouble with https://jasik.xyz/resume.pdf. When I run it through the blc package itself it gives the error
|
Ah so it's erroring because of the mime type ok thanks |
https://github.com/jasikpark/jasik-xyz/runs/1073175123?check_suite_focus=true
I was testing out #7 and ran into this problem. I'm not sure what's wrong, I guess I'll try running it again?
The text was updated successfully, but these errors were encountered: