ci: Publish pg_search to Postgres.app via PR from paradedb-bot#4913
Open
philippemnoel wants to merge 18 commits into
Open
ci: Publish pg_search to Postgres.app via PR from paradedb-bot#4913philippemnoel wants to merge 18 commits into
philippemnoel wants to merge 18 commits into
Conversation
abf86f3 to
c74c59b
Compare
Member
Author
|
See here for workflow testing: https://github.com/paradedb/paradedb/releases/tag/v0.0.0 |
3 tasks
philippemnoel
commented
Apr 30, 2026
philippemnoel
commented
Apr 30, 2026
philippemnoel
commented
Apr 30, 2026
mdashti
approved these changes
May 2, 2026
Adds a workflow that builds a Postgres.app installer .pkg for pg_search on each release tag, uploads it to the GitHub Release, and opens a PR from paradedb-bot's fork to PostgresApp/PostgresApp updating the download link on https://postgresapp.com/extensions/. Any previous open PR from the bot is closed first so only the latest release is pending review upstream. See PostgresApp/PostgresApp#799 for context.
- Clone PostgresApp/Extensions and reuse its Resources/, Scripts/, and distribution.xml so the pkg matches what Postgres.app maintainers ship for other extensions (codesign/notarize still skipped pending cert). - Switch the upstream PR target from docs/extensions/index.md regex edits to a clean JSON update at docs/_data/extensions/<pg>/pg_search.json, per Jakob's note that download metadata now lives there. - Use the upstream pkg identifier (com.postgresapp.extension.<pg>.<ext>) and naming (pg_search-pg<pg>-<ver>.pkg) for parity with other Postgres.app extensions.
b492b77 to
5716365
Compare
Member
Author
|
This is waiting on Apple to approve our developer account |
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
Closes #4630
.github/workflows/publish-pg_search-postgresapp.ymlthat builds a Postgres.app installer.pkgfor pg_search on each release tag (PG 18 only — earlier versions don't support extensions installed in Application Support).v<version>and namedpg_search-pg18-<version>.pkg, matching the naming convention used by PostgresApp/Extensions.Resources/,Scripts/,distribution.xml) cloned fromPostgresApp/Extensionsat workflow time, so the installer layout, identifiers, install location, and pre/post-install behavior match what Postgres.app maintainers ship for other extensions.paradedb-bot's fork to PostgresApp/PostgresApp updatingversionanddownload_urlindocs/_data/extensions/18/pg_search.json(the source for https://postgresapp.com/extensions/), per Jakob's guidance that download metadata is now stored as JSON.paradedb-botupstream is closed with a "Superseded" comment before the new one is opened, so only the latest release is ever pending review.See PostgresApp/PostgresApp#799 for the upstream conversation that motivated this.
Notes
publish-pg_search-macos.yml. Upstream'sbuild.zshrunscodesignagainstDeveloper ID Applicationand usesnotarytoolwith apostgresappkeychain profile; we skip both because we don't have an Apple Developer cert wired up in CI yet. Installers will trigger a Gatekeeper warning on first install. Apple Developer codesigning +notarytoolcan be added later when the secrets are configured.paradedb-botdoes not yet have a fork ofPostgresApp/PostgresApp—gh repo forkcreates it on first run.PARADEDB_BOT_GITHUB_TOKENneedspublic_reposcope to fork and open PRs against the public upstream (already required by other publish workflows).Test plan
workflow_dispatchagainst an existing release tag and confirm the pkg is uploaded to the release.PostgresApp/PostgresAppfromparadedb-bot:update-pg_search-<version>-pg18updatingversionanddownload_urlindocs/_data/extensions/18/pg_search.json..pkgon a Mac with Postgres.app PG 18 and runCREATE EXTENSION pg_search;to confirm the layout is correct.