Skip to content

Add oxipng to pre-commit#1669

Closed
nikoshell wants to merge 5 commits into
ep2026from
ep2026-images
Closed

Add oxipng to pre-commit#1669
nikoshell wants to merge 5 commits into
ep2026from
ep2026-images

Conversation

@nikoshell

@nikoshell nikoshell commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

CLOSE #1519
FIX #1518

@europython-society

europython-society commented Jun 23, 2026

Copy link
Copy Markdown
Member

Preview available

Key Value
url https://ep2026-images.ep-preview.click
last update 2026-06-23T15:14:54.256Z

@nikoshell nikoshell self-assigned this Jun 23, 2026

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

Comment thread .pre-commit-config.yaml
Comment on lines +23 to +24
- repo: https://github.com/shssoichiro/oxipng
rev: v9.1.4

@hugovk hugovk Jun 24, 2026

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.

Suggested change
- repo: https://github.com/shssoichiro/oxipng
rev: v9.1.4
- repo: https://github.com/oxipng/oxipng
rev: v10.1.1

Comment thread .pre-commit-config.yaml
rev: v9.1.4
hooks:
- id: oxipng
args: ["-o", "2", "--strip", "all"]

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.

-o 2 is the default

Suggested change
args: ["-o", "2", "--strip", "all"]
args: ["--strip=all"]

@nikoshell

Copy link
Copy Markdown
Contributor Author

Remember the README changes from #1519:

https://github.com/EuroPython/website/pull/1519/changes#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

I'm sorry I totally missed that you already add .pre-commit-config.yaml to that PR. I was thinking that we do optimization manually again.

I'm closing this one and we can merge #1519.

@nikoshell nikoshell closed this Jun 24, 2026
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.

Image sizes make cloning the project slow

3 participants