ci: update workflow and readme#56
Merged
wuziqian211 merged 2 commits intohappycola233:mainfrom Jan 12, 2026
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the CI/CD workflows to use ditto for proper macOS app packaging while preserving metadata, and adds macOS-specific installation instructions to the README. The changes improve formatting consistency across documentation and modernize workflow configurations.
Changes:
- Updated PyInstaller spec to enable stripping and disable UPX for macOS builds
- Added macOS installation warning about quarantine attributes and proper app location
- Modernized GitHub Actions workflows with consistent formatting and better step separation
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tchMaterial-parser.spec | Changed macOS build settings: enabled stripping and disabled UPX compression |
| README.md | Added spacing to section headers, included macOS-specific installation instructions with xattr command |
| .github/workflows/python-app.yml | Updated to actions/checkout@v6, Python 3.14, and improved formatting |
| .github/workflows/build-release.yml | Updated actions versions, improved step separation, replaced zip with ditto for macOS packaging |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
Author
|
@wuziqian211 @happycola233 Please review this PR, and probably a new version is needed to ensure the app from actions is runnable. |
wuziqian211
approved these changes
Jan 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fully tested with new code in https://github.com/Mister-Hope/tchMaterial-parser/releases/tag/v3.3-test2
Now workflow uses ditto to keep metadata properly while packing.
A guideline is added to README.md to explain how to make the Mac app work.
workflow config is refined for better readability
upx is disabled in config
Though the Mac's latest runner does not have upx so py-installer won't use it, it can be problematic in some cases; our app is not large enough for introducing a compressor.
Note
I would suggest enabling stripping in other builds, but since I can not test them, I will leave them here.