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
Remove documentation for wptrunner as a standalone project #28806
Conversation
fe0572c
to
6584210
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The installing part is definitely wrong/outdated, but it's possible that the usage parts aren't really documented elsewhere. Could be worth saving some of that if you have time.
@jgraham I'll comment on the bits that I thought you'd tell me to move somewhere else :) |
wptrunner 1.14 was released in 2015, and while it can be installed in a virtualenv, these instructions should not be followed by anyone these days. Integrate some of it into from-local-system.md instead.
6584210
to
de157b6
Compare
@jgraham do you want to review the salvaged (half rewritten) documentation? |
@@ -152,7 +152,7 @@ customising the test run: | |||
./wpt run --help | |||
|
|||
[A complete listing of the command-line arguments is available | |||
here](command-line-arguments.md). | |||
here](command-line-arguments.md#run). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked the build output and this actually broke stuff:
-<p><a class="reference internal" href="command-line-arguments.html"><span class="doc">A complete listing of the command-line arguments is available
-here</span></a>.</p>
+<p><a class="reference external" href="command-line-arguments.md#run">A complete listing of the command-line arguments is available
+here</a>.</p>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found there are a few cases of broken links like this already that also need fixing...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sent #28831, will apply the same fix here.
wptrunner 1.14 was released in 2015, and while it can be installed in a
virtualenv, these instructions should not be followed by anyone these
days.
Integrate some of it into from-local-system.md instead.