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

Downgrade ttfunk #1374

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dacook
Copy link

@dacook dacook commented Mar 2, 2025

The issue in ttfunk remains unsolved, meaning this version of Prawn is also broken, and has been for some time. This wastes time for each user of the gem, having to debug and research the the problem until they find some clues in issue comments and try those.

So I suggest that Prawn reverts to the previous version to make it work out of the box. Is it this simple?

 Skips ttfunk 1.8 until issue prawnpdf/ttfunk#102 fixed
@dacook dacook marked this pull request as draft March 2, 2025 22:41
@dacook dacook marked this pull request as ready for review March 2, 2025 22:52
@dacook
Copy link
Author

dacook commented Mar 2, 2025

The CI failure may be due to the gem cache. I was able to bundle install fine, but it seems that CI is relying solely on cached gems.

Co-Pilot makes a good point:

The failing job in the CI workflow appears to be related to the cache action for the gems. The cache hit indicates that it successfully restored the cache but the subsequent steps might be failing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant