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

Improve test for rustdoc::bare_urls lint #84079

Merged
merged 2 commits into from
Apr 12, 2021

Conversation

camelid
Copy link
Member

@camelid camelid commented Apr 11, 2021

  • Rename url-improvements test to bare-urls
  • Run rustfix for bare-urls test

camelid added 2 commits April 10, 2021 18:37
The lint used to be called `non-autolinks`, and linted more than just
bare URLs. Now, it is called `bare-urls` and only lints against bare
URLs. So, `bare-urls` is a better name for the test.
This will help us ensure that it emits valid suggestions.
@camelid camelid added A-testsuite Area: The testsuite used to check the correctness of rustc T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Apr 11, 2021
@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 11, 2021
@camelid
Copy link
Member Author

camelid commented Apr 11, 2021

r? @jyn514

@jyn514
Copy link
Member

jyn514 commented Apr 11, 2021

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Apr 11, 2021

📌 Commit aabc363 has been approved by jyn514

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 11, 2021
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Apr 11, 2021
…yn514

Improve test for `rustdoc::bare_urls` lint

- Rename `url-improvements` test to `bare-urls`
- Run rustfix for `bare-urls` test
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 12, 2021
Rollup of 7 pull requests

Successful merges:

 - rust-lang#83669 (Issue 81508 fix)
 - rust-lang#84014 (Improve trait/impl method discrepancy errors)
 - rust-lang#84059 (Bump libc dependency of std to 0.2.93)
 - rust-lang#84067 (clean up example on read_to_string)
 - rust-lang#84079 (Improve test for `rustdoc::bare_urls` lint)
 - rust-lang#84094 (Remove FixedSizeArray)
 - rust-lang#84101 (rustdoc: Move crate loader to collect_intra_doc_links::early )

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit ae549b5 into rust-lang:master Apr 12, 2021
@rustbot rustbot added this to the 1.53.0 milestone Apr 12, 2021
@camelid camelid deleted the improve-bare-urls-test branch April 12, 2021 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants