search

cynkra / cynkrablog

Source of the cynkra blog
https://cynkra.com/blog
0 stars 1 forks source link

Save lychee cache only when there are no failures #44

Closed maelle closed 8 months ago

maelle commented 8 months ago

An idea to tackle https://github.com/cynkra/cynkrablog/pull/42#issuecomment-1891648308

There were false positives in the PR. To avoid them, I only knew one solution: opening another PR with the same changes.

maelle commented 8 months ago

@pat-s do you have an opinion/tips on this? Currently if lychee identifies an URL as not working, because of the cache it keeps considering them not working, which is difficult to handle in a PR where an URL first was not working for whatever reasons.