-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
rustdoc tests error output without tidy
is very bad
#85061
Comments
"depend" is not quite correct - tests will pass fine without it, but if they fail the error message is very bad. |
We improved the error message in case |
I think we can close it since it's possible to run rustdoc tests without |
@GuillaumeGomez this problem is still not solved. |
Renaming the issue to reflect this then. |
tidy
which isn't installed on release builderstidy
is very bad
@tmandry A simple solution is to just install tidy on the release builders. I don't know how to do that but it's packaged in nearly every distro. |
at the very least the "tidy is not installed" message should link to the tool's homepage so it can be installed. this is especially confusing because of the existence of the tidy tool. |
Note:
tidy
here is the html diff / format tool, notsrc/tools/tidy
From the 1.52 retrospective: Having this installed would have made it much easier to spot the issue in #84950.
The text was updated successfully, but these errors were encountered: