fix: exclude /donate from broken link checker

[mcdurdin]

/donate redirects to another site that gives a 403 forbidden response to bots, which breaks the link checker. So just exclude it.

Fixes: #488

[darcywong00]

lgtm assuming the CI link check passes

[mcdurdin]

Hmm, turns out that there are 4 references to broken-link-checker. I can understand 2 of them, and no obvious way to consolidate those, but not sure why we have the ones in composer.json as well, would recommend removing them?

[darcywong00]

but not sure why we have the ones in composer.json as well, would recommend removing them?

Ah, I added the one in composer.json in #389 for feature parity with help.keyman.com

I suppose we can remove in a separate PR

[mcdurdin]

Ah, I added the one in composer.json in #389 for feature parity with help.keyman.com

Hmm, so we should probably remove from all then. Let's keep composer.json minimal, remove all scripts, seeing as we do them in ci.yml now?

I see we never call composer run test either, which means we should probably add vendor/bin/phpunit --testdox to ci.yml and/or Dockerfile?

How do we run lint and phpunit tests locally?

[darcywong00]

I see we never call composer run test either, which means we should probably add vendor/bin/phpunit --testdox to ci.yml and/or Dockerfile?

How do we run lint and phpunit tests locally?

Looks like a TODO 😞

keyman.com/build.sh

Line 31 in 03025d8

# TODO: lint tests

[mcdurdin]

Okay, added #493 to track composer.json/test changes