Skip to content

Remove unused images#649

Merged
aduth merged 1 commit intomainfrom
aduth-rm-unused-images
Dec 12, 2024
Merged

Remove unused images#649
aduth merged 1 commit intomainfrom
aduth-rm-unused-images

Conversation

@aduth
Copy link
Copy Markdown
Contributor

@aduth aduth commented Dec 12, 2024

🛠 Summary of changes

Removes unused images from source code.

Discovered with a similar script as in 18F/identity-idp#11215:

for i in `ls images`
do
  if ! grep -q -R _articles -e $i
  then
    echo $i;
  fi
done

declare-incident-workflow-1.png: Added in #418, never used
declare-incident-workflow-2.png: Added in #418, never used
declare-incident-workflow-3.png: Added in #418, never used
idv-session-errors-failure.png: Added in #471, never used
idv-ssn-rate-limited.png: Added in #471, never used
statuspage-sample-message.png: Last reference removed in #389

📜 Testing Plan

Verify removed images aren't referenced anywhere.

Copy link
Copy Markdown
Contributor

@vrajmohan vrajmohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@aduth aduth merged commit 2ec60a1 into main Dec 12, 2024
@aduth aduth deleted the aduth-rm-unused-images branch December 12, 2024 18:43
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