Remove unused imports in code base

CommentFileSizeAuthor
#7 unused_imports_in_code-2687639-7.patch2.51 KBnerdstein
#4 unused_imports_in_code-2687639-4.patch2.83 KBmarkdorison
#3 2687639-2.patch2.82 KBheykarthikwithu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heykarthikwithu created an issue. See original summary.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
Assigned: heykarthikwithu » Unassigned
Status: Active » Needs review
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
FileSize
2687639-2.patch2.82 KB
markdorison’s picture

markdorison
he/him
CreditAttribution: markdorison at Chromatic commented
FileSize
unused_imports_in_code-2687639-4.patch2.83 KB

Identified and removed a couple additional instances of 'unused use statements'.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +php7 
+++ b/password_policy.module
@@ -1,8 +1,5 @@
-use Drupal\Component\Utility\String;

yay!

nerdstein’s picture

nerdstein
he/him
Primary language English
Location United States
CreditAttribution: nerdstein at CivicActions commented
Assigned: Unassigned » nerdstein

Reviewed the patch in #4 and will be committing this weekend. Thanks, all.

nerdstein’s picture

nerdstein
he/him
Primary language English
Location United States
CreditAttribution: nerdstein at CivicActions commented
FileSize
unused_imports_in_code-2687639-7.patch2.51 KB

The patch needs rebuilt from recent commits...

nerdstein’s picture

nerdstein
he/him
Primary language English
Location United States
CreditAttribution: nerdstein at CivicActions commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.