Skip to content

Fix automated release process: Updates GHA types and build dependencies#1675

Merged
tonsky merged 5 commits intotonsky:masterfrom
hunterhogan:releasesCode
Jan 19, 2026
Merged

Fix automated release process: Updates GHA types and build dependencies#1675
tonsky merged 5 commits intotonsky:masterfrom
hunterhogan:releasesCode

Conversation

@hunterhogan
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

@hunterhogan hunterhogan left a comment

Choose a reason for hiding this comment

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

Tested the automated release creation files on my fork. A seemingly valid zip was created.

In the GitHub action, I emulated "bootstrap_macos.sh" instead of trying to build a docker container.

In a few places, some code effectively suppressed error reporting. I removed those suppressors so I could see what was happening, but I didn't restore them.

The code was AI assisted (Gemini 3 Pro preview via Copilot). Nevertheless, I have read every line of the code, and I tested it on my account first.

Thank you for making a great font.

Copy link
Copy Markdown
Owner

@tonsky tonsky left a comment

Choose a reason for hiding this comment

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

Thanks! Mostly looks fine, but I would like to understand what does contents: write do. I don't think I ever needed it in any of my repos

jobs:
build:
permissions:
contents: write
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What does this do?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My understanding is that it is used to create the artifacts. I only have four workflows and they are all here. https://github.com/hunterhogan/hunterMakesPy/tree/main/.github/workflows The two that create files have write permissions, but the other two only have read.

@tonsky tonsky merged commit 9342668 into tonsky:master Jan 19, 2026
@tonsky
Copy link
Copy Markdown
Owner

tonsky commented Jan 19, 2026

It's been a minute, I’m not sure how this build used to work, and Github Actions backwards compatibility keeps us all busy. Thanks for bringing it up to date!

@hunterhogan hunterhogan deleted the releasesCode branch January 19, 2026 03:05
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