Skip to content
This repository was archived by the owner on Mar 15, 2024. It is now read-only.

[BEEEP] Add script for optimizing images - #1374

Merged
Hinton merged 6 commits into
masterfrom
feature/optimize-images
Jan 6, 2022
Merged

[BEEEP] Add script for optimizing images#1374
Hinton merged 6 commits into
masterfrom
feature/optimize-images

Conversation

@Hinton

@Hinton Hinton commented Jan 6, 2022

Copy link
Copy Markdown
Member

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

I got tired of having un-minified images in the repo. So I wrote together a script that uses squoosh to minimize them, and added it to lint-staged.

Before you submit

  • I have checked for linting errors (npm run lint) (required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@Hinton Hinton changed the title Add script for optimizing images [BEEEP] Add script for optimizing images Jan 6, 2022
@Hinton
Hinton marked this pull request as ready for review January 6, 2022 18:49
@Hinton
Hinton requested a review from a team January 6, 2022 18:49

@MGibson1 MGibson1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's lose the powershell and just have it part of the linter.

@Hinton
Hinton requested a review from MGibson1 January 6, 2022 19:59
@Hinton
Hinton merged commit 2f5d020 into master Jan 6, 2022
@Hinton
Hinton deleted the feature/optimize-images branch January 6, 2022 20:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants