Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install pngquant in MkDocs Pipeline for correct image optimization #112

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

jlondonobo
Copy link
Contributor

Overview

The third time's the charm (hopefully). Following the recommendation from the optimize documentation, this PR ensures pngquant is installed prior to building the instructor documentation.

I apologize for the iterative approach. Because I have no access to MkDocs insiders, I couldn't replicate instructor's precise workflow to test it comprehensively in one go.

Changes

  • Install pngquant before building instructor's documentation.

Tests

Tested the installation of pngquant in this fork.

@jxnl
Copy link
Collaborator

jxnl commented Sep 25, 2023

omg thanks so much for this i've been super busy and its a gift to see these PRs.

@jxnl jxnl merged commit 0f896fe into instructor-ai:main Sep 25, 2023
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