Skip to content

Comments

subject_name: fix stale unused_imports allow.#171

Merged
cpu merged 1 commit intorustls:mainfrom
cpu:cpu-forgets-todos-and-feels-bad
Sep 8, 2023
Merged

subject_name: fix stale unused_imports allow.#171
cpu merged 1 commit intorustls:mainfrom
cpu:cpu-forgets-todos-and-feels-bad

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
@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