chore: use Lychee pre-commit hook

[agriyakhetarpal]

CI time is not a concern for our website, so even if our linter takes thirty seconds instead of five, that should be fine

[netlify]

✅ Deploy Preview for pybamm-developer-preview ready!

Name	Link
🔨 Latest commit	5ed54d9
🔍 Latest deploy log	https://app.netlify.com/sites/pybamm-developer-preview/deploys/66e0390452e8680008fb7b13
😎 Deploy Preview	https://deploy-preview-193--pybamm-developer-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[agriyakhetarpal]

Strange, works fine locally

[arjxn-py]

Thanks @agriyakhetarpal, it also doesn't show that for which specific link it is failing for. Is it something it'll show like previous workflow?

[agriyakhetarpal]

Yes, it would show which link it would fail for, but it doesn't run with https://pre-commit.ci, so the links were not checked with this commit.

[agriyakhetarpal]

https://pre-commit.com/#rust

Looks like this is an issue worth reporting in lycheeverse/lychee upstream: it works with pre-commit locally and would do so with a GHA setup, but it doesn't work with https://pre-commit.ci. I'm not sure if I have the time to prepare a proper report and investigate right now. Would you like to do that, @arjxn-py?

[arjxn-py]

https://pre-commit.com/#rust

Looks like this is an issue worth reporting in lycheeverse/lychee upstream: it works with pre-commit locally and would do so with a GHA setup, but it doesn't work with https://pre-commit.ci. I'm not sure if I have the time to prepare a proper report and investigate right now. Would you like to do that, @arjxn-py?

Sure @agriyakhetarpal, thanks for pinging. Would be happy to do that.