-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Tracking issue for moving rustdoc* tests to appropriate subdirs #136501
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
Comments
@rustbot claim |
@yegeunyang could you hold off on this until #136400 is merged? otherwise whichever PR isn't merged first will have a bunch of annoying merge conflicts. |
Of course! |
@rustbot release-assignment |
additionally, there's several tests involving deref that could get moved to |
Skimmed the rustdoc-ui tests and put together hopefully a more concrete list of files that fall under the scope of this issue. I wouldn't mind making the changes once #136400 is merged. Lints Intra doc Deref |
@jusexton there's also |
another: |
Wasn't able to find this file. This is located under
Good catch, added to the complete list. |
That's because I added it in a PR that hasn't been merged yet 🤦 It's been so long since I started the PR I convinced myself it was an existing file I just added to. I can just fix that one myself. |
another one: |
@rustbot claim |
waiting on #136400 |
@Evalir PR is merged! |
awesome thank you for the ping! i'll get to this soon! |
Found another one: |
Feel free to split this up into several smaller PRs if you want, it should be less likely to get hit with multiple merge conflicts that way. |
Uh oh!
There was an error while loading. Please reload this page.
Lots of tests could be moved into an existing subdir to be categorized more accurately.
Tests are grouped first by testsuite, then by where they should be moved to.
rustdoc-ui
rustdoc-ui/lints/
rustdoc-ui/intra-doc
rustdoc-ui/deref
rustdoc
rustdoc/doc-cfg
The text was updated successfully, but these errors were encountered: