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

subject_name: fix stale unused_imports allow. #171

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

cpu
Copy link
Member

@cpu cpu commented Sep 8, 2023

Noticed this while backporting various changes for #170.

My kingdom for a clippy lint that can tell when unused_imports allows aren't needed anymore 😩

@cpu cpu self-assigned this Sep 8, 2023
@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #171 (d54e874) into main (7ff3664) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #171   +/-   ##
=======================================
  Coverage   96.64%   96.64%           
=======================================
  Files          16       16           
  Lines        4532     4532           
=======================================
  Hits         4380     4380           
  Misses        152      152           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

📢 Have feedback on the report? Share it here.

@cpu cpu added this pull request to the merge queue Sep 8, 2023
Merged via the queue into rustls:main with commit 64b97f3 Sep 8, 2023
24 checks passed
@cpu cpu deleted the cpu-forgets-todos-and-feels-bad branch September 8, 2023 16:32
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.

2 participants