set xsi to http instead of https

[ebbit1q]

fixes #202

[tooomm]

Nice way with the lycheeignore file. 👍

Alternative would be to add --exclude www.w3.org or similar to the args.

[ebbit1q]

this was the way to exclude urls listed in the action's readme, if you prefer the arg that'd also be fine, it'd keep the workflow confined to its own file instead of cluttering the repo.

[tooomm]

Probably there won't be any other links added there, right?
In that case --exclude seems cleaner.

Potential rework of excluding URLs pending

[tooomm]

I guess the continue-on-error behaves different than you think and it's not helpful here:
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idcontinue-on-error

We only have a single job that is run (check_urls)...

But just saw that you wanted to get rid of the if: always() that way.
Even putting it on the step level would only make the workflow pass, but still skip following steps I think. Meaning if the check on tokens.xml fails, challenge_tokens.xml is never checked. It's probably ok, to only put it on the second check - just for consistency I put it to both.
Or did you test this?

[ebbit1q]

hmm, I might have misinterpreted the docs then, why don't we have one step checking both files? this would marginally improve speed too.

[ebbit1q]

👍

[tooomm]

Good idea to combine the two checks in one run!

Now the env is less useful, but it still makes it a bit more readable I think.