Skip to content

Conversation

@Calinou
Copy link
Member

@Calinou Calinou commented Nov 7, 2023

  • Exclude all images in the top-level img/ folder.
  • Take WebP images into account.
  • Fix shellcheck warnings.
  • Print message with destination path of the list of unused images.
  • Mark the script as executable so it can be run more easily.

@mhilbrunner
Copy link
Member

mhilbrunner commented Nov 9, 2023

@Calinou Not sure if you think it worthwhile to switch to -iname as discussed above, otherwise this looks ready to merge. :)

- Exclude all images in the top-level `img/` folder.
- Take WebP images into account.
- Fix shellcheck warnings.
- Mark the script as executable so it can be run more easily.
@Calinou Calinou force-pushed the update-unused-images-script branch from ed07fd8 to 49f1f01 Compare November 13, 2023 20:52
@Calinou Calinou mentioned this pull request Nov 13, 2023
@mhilbrunner mhilbrunner merged commit 70c6bca into godotengine:master Nov 14, 2023
@mhilbrunner
Copy link
Member

Thanks!

@Calinou Calinou deleted the update-unused-images-script branch November 18, 2023 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants