Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate from liche to lychee #123

Merged
merged 5 commits into from
Jun 10, 2021
Merged

Migrate from liche to lychee #123

merged 5 commits into from
Jun 10, 2021

Conversation

MichaIng
Copy link
Owner

@MichaIng MichaIng commented Jun 9, 2021

This requires to spin up a local webserver to be able to check internal links. The GitHub actions virtual machines come with Apache2 and Nginx on board, so we can configure and start Nginx.

Additionally migrate to new Java setup action and version for v.Nu.

This requires to spin up a local webserver to be able to check internal links. The GitHub actions virtual machines come with Apache2 and Nginx on board, so we can configure and start Nginx.

Additionally migrate to new Java setup action and version for v.Nu.
@MichaIng MichaIng added the enhancement New feature or request label Jun 9, 2021
until next lychee release which contains lycheeverse/lychee#236.

Try to fix verbose flag by moving it before options.
@MichaIng
Copy link
Owner Author

MichaIng commented Jun 9, 2021

Wow, this tools is extremely fast. 2 seconds, all checks passed. Somehow the verbose flag seems to be ignored if there is no regular console on STDIN/STDOUT (my guess), but I verified here that it really finds all links in all files.

Compared to liche, taking minutes to finally fail with timeouts. And it became more important now to do the migration, since recently liche gets killed by the action, due to too high memory usage, if I got the exit code correctly. Will do the migration on Docs as well.

It is ignored when STDOUT and STDERR are redirected (not attached to a regular terminal)
@MichaIng MichaIng merged commit d56ec1a into dev Jun 10, 2021
@MichaIng MichaIng deleted the dev-MichaIng-lychee branch June 10, 2021 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant