Skip to content

Fix skiplinkcheck #638

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

Merged
merged 1 commit into from
Feb 25, 2019
Merged

Fix skiplinkcheck #638

merged 1 commit into from
Feb 25, 2019

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Feb 22, 2019

--skiplinkcheck and --linkcheckonly were documented as only working
with --doc but they are actually only used with --all. This fixes
the documentation and the preflight checks to reflect that reality.

`--skiplinkcheck` and `--linkcheckonly` were documented as only working
with `--doc` but they are actually only used with `--all`. This fixes
the documentation and the preflight checks to reflect that reality.
@nik9000 nik9000 requested review from a user and ddillinger February 22, 2019 22:13
@lcawl
Copy link
Contributor

lcawl commented Feb 22, 2019

Good catch! Makes sense, since we only do the link checks on --all

@nik9000
Copy link
Member Author

nik9000 commented Feb 22, 2019

Good catch! Makes sense, since we only do the link checks on --all

I bumped into this yesterday when I tried to skip the links check!

@nik9000 nik9000 merged commit cab50cc into elastic:master Feb 25, 2019
@nik9000
Copy link
Member Author

nik9000 commented Feb 25, 2019

Thanks for looking @ddillinger and @lcawl!

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.

3 participants