Markdown link check is failing

[SimenB]

https://github.com/jest-community/eslint-plugin-jest/actions/workflows/markdown-link-check.yml

@G-Rath do you have any idea what's going on here? 😅 We might wanna use (and possibly migrate to the option recommended in its readme?)

No context for why it fails... 🤔

Running it locally hangs forever (or, at least 5 minutes, at which point I killed it)

[G-Rath]

yeah I've been trying to dig into it locally and getting the same 123 exit code without any other details - for me it does seem to work, taking 1-2 minutes, but no matter what I can't seem to get it actually tell me what it has a problem with no matter what I do with the flags.

Over the holiday period somehow I managed to determine it was unhappy with one of the eslint docs that did seem to be broken but now seems to be right again? I was going to try looking further into it today, but was also considering just disabling it for now and tackling it next month after getting through some other stuff on my plate 🤷

this is the output I can get

(where `script.sh` is just a copy and paste of the bash from the workflow)

eslint-plugin-jest on  main [$?] is 📦 v28.10.0 via  v20.11.0
❯ bash script.sh

FILE: ./README.md

FILE: ./docs/rules/no-restricted-jest-methods.md

FILE: ./docs/rules/require-top-level-describe.md

FILE: ./docs/rules/prefer-called-with.md

FILE: ./docs/rules/no-confusing-set-timeout.md

FILE: ./docs/rules/no-disabled-tests.md

FILE: ./docs/rules/prefer-spy-on.md

FILE: ./docs/rules/prefer-comparison-matcher.md

FILE: ./docs/rules/prefer-expect-assertions.md

FILE: ./docs/rules/prefer-expect-resolves.md

FILE: ./docs/rules/padding-around-test-blocks.md

FILE: ./docs/rules/no-hooks.md

FILE: ./docs/rules/prefer-each.md

FILE: ./docs/rules/max-expects.md

FILE: ./docs/rules/prefer-hooks-on-top.md

FILE: ./docs/rules/no-restricted-matchers.md

FILE: ./docs/rules/no-identical-title.md

FILE: ./docs/rules/no-untyped-mock-factory.md

FILE: ./docs/rules/no-duplicate-hooks.md

FILE: ./docs/rules/no-done-callback.md

FILE: ./docs/rules/prefer-strict-equal.md

FILE: ./docs/rules/no-conditional-in-test.md

FILE: ./docs/rules/no-test-return-statement.md

FILE: ./docs/rules/expect-expect.md

FILE: ./docs/rules/no-conditional-expect.md

FILE: ./docs/rules/consistent-test-it.md

FILE: ./docs/rules/no-focused-tests.md

FILE: ./docs/rules/prefer-equality-matcher.md

FILE: ./docs/rules/no-mocks-import.md

FILE: ./docs/rules/no-large-snapshots.md

FILE: ./docs/rules/no-commented-out-tests.md

FILE: ./docs/rules/no-interpolation-in-snapshots.md

FILE: ./docs/rules/prefer-hooks-in-order.md

FILE: ./docs/rules/padding-around-all.md

FILE: ./docs/rules/valid-describe-callback.md

FILE: ./docs/rules/no-deprecated-functions.md

FILE: ./docs/rules/no-jasmine-globals.md

FILE: ./docs/rules/require-hook.md

FILE: ./docs/rules/padding-around-after-all-blocks.md

FILE: ./docs/rules/prefer-mock-promise-shorthand.md

FILE: ./docs/rules/prefer-importing-jest-globals.md

FILE: ./docs/rules/prefer-todo.md

FILE: ./docs/rules/valid-expect.md

FILE: ./docs/rules/no-export.md

FILE: ./docs/rules/no-standalone-expect.md

FILE: ./docs/rules/max-nested-describe.md

FILE: ./docs/rules/no-alias-methods.md

FILE: ./docs/rules/no-test-prefixes.md

FILE: ./docs/rules/valid-expect-in-promise.md

FILE: ./docs/rules/prefer-to-have-length.md

FILE: ./docs/rules/padding-around-expect-groups.md

FILE: ./docs/rules/prefer-snapshot-hint.md

FILE: ./docs/rules/prefer-lowercase-title.md

FILE: ./docs/rules/require-to-throw-message.md

FILE: ./docs/rules/padding-around-describe-blocks.md

FILE: ./docs/rules/padding-around-before-all-blocks.md

FILE: ./docs/rules/padding-around-after-each-blocks.md

FILE: ./docs/rules/prefer-to-contain.md

FILE: ./docs/rules/prefer-jest-mocked.md

FILE: ./docs/rules/valid-title.md

FILE: ./docs/rules/padding-around-before-each-blocks.md

FILE: ./docs/rules/unbound-method.md

FILE: ./docs/rules/prefer-to-be.md

[G-Rath]

looks like a bug in the checker:

eslint-plugin-jest on  main [$?] is 📦 v28.10.0 via  v20.11.0
❯ yarn markdown-link-check -c markdown_link_check_config.json -v docs/rules/valid-title.md

FILE: docs/rules/valid-title.md
docs/rules/valid-title.md is not ok! TypeError: Cannot read properties of undefined (reading 'map')
    at runMarkdownLinkCheck (/home/jones/workspace/projects-oss/eslint-plugin-jest/node_modules/markdown-link-check/markdown-link-check:302:24)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async processInput (/home/jones/workspace/projects-oss/eslint-plugin-jest/node_modules/markdown-link-check/markdown-link-check:294:5)
    at async main (/home/jones/workspace/projects-oss/eslint-plugin-jest/node_modules/markdown-link-check/markdown-link-check:318:13)
we are not ok!

[SimenB]

I somehow failed to paste the link I alluded to in the OP: https://github.com/gaurav-nelson/github-action-markdown-link-check

[github-actions]

🎉 This issue has been resolved in version 28.10.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀