Prepare PreviewMD for open source - #16
Merged
Merged
Conversation
Signed-off-by: Adam Jesionkiewicz <adam@jesion.pl>
ashtree74
commented
Aug 12, 2026
ashtree74
left a comment
Owner
Author
There was a problem hiding this comment.
Self-review completed with no blocking findings.
Checked:
- final scope contains only the open-source preparation work
- Apache-2.0, NOTICE, authorship, governance, DCO, and trademark language are internally consistent
- Astrography appears only as the current macOS signing/distribution entity, not as owner or governor
- assembled app packaging carries LICENSE, NOTICE, and third-party notices
- CI uses read-only permissions and a pinned checkout action
- working tree and all reachable Git history passed secret scans
- no generated app, database, subscriber export, signing material, or credential file is included
Residual low-risk findings remain explicitly recorded in docs/PUBLICATION_AUDIT.md: historical release binaries increase clone size, and the landing page retains third-party analytics JavaScript.
This is the bootstrap PR for its own CI and CODEOWNERS policy. The author cannot approve their own PR, so after required checks pass it will use the repository's PR-only founder bypass, preserving the PR and audit trail.
Signed-off-by: Adam Jesionkiewicz <adam@jesion.pl>
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.
Summary
Verification
swift test— 135 XCTest tests and 5 Quick Look Swift Testing tests passedpython3 -m unittest discover -s site -p 'test_*.py'— 9 tests passed./scripts/build-app.sh— app and Quick Look extension built Universal 2 (x86_64+arm64)git diff --check, working-tree secret scan, and full-historygit-secretsscan passedPublication notes
The repository remains private until this PR is reviewed and merged. The new required CI contexts and CODEOWNERS file are introduced by this PR, so the existing PR-only founder bypass may be needed for this one bootstrap merge. Subsequent pull requests will receive the checks normally.
The audit found no credentials, signing material, subscriber data, or publication blocker. Residual low-risk findings are documented in docs/PUBLICATION_AUDIT.md.