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

ci: check links #54

Merged
merged 5 commits into from
Aug 5, 2020
Merged

ci: check links #54

merged 5 commits into from
Aug 5, 2020

Conversation

ouuan
Copy link
Member

@ouuan ouuan commented Aug 5, 2020

Now the links are checked in the build test.

It seems that it sometimes fails on too many requests (peter-evans/link-checker#29). If it happens in the future, we can add -x 'https?://' in the arguments.

@ouuan ouuan requested a review from coder3101 August 5, 2020 13:36
@render
Copy link

render bot commented Aug 5, 2020

@pull-assistant
Copy link

pull-assistant bot commented Aug 5, 2020

Score: 0.97

Best reviewed: commit by commit


Optimal code review plan

     ci: check links

     ci(check-link): set root directory

     ci(check-link): fail on errors

     ci(check-link): disable verbose

     fix(contributing): fix link to build from source

Powered by Pull Assistant. Last update 3c81fc8 ... c9bc69f. Read the comment docs.

@coder3101 coder3101 merged commit 81ce85a into hugo Aug 5, 2020
@coder3101 coder3101 deleted the check-links branch August 5, 2020 14:08
github-actions bot pushed a commit that referenced this pull request Aug 5, 2020
* ci: check links

* ci(check-link): set root directory

* ci(check-link): fail on errors

* ci(check-link): disable verbose

* fix(contributing): fix link to build from source 81ce85a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants