Skip to content

Conversation

@pyramation
Copy link
Contributor

@pyramation pyramation commented Jan 2, 2026

Summary

Removes the pyramation/node-sqitch:20.12.0 container from CI workflows, allowing jobs to run directly on ubuntu-latest. This simplifies the CI setup as the custom container is no longer needed.

Changes:

  • Removed container: pyramation/node-sqitch:20.12.0 directive from both workflows
  • Fixed PGHOST from pg_db to localhost (required since jobs now run directly on the runner, not inside a container)
  • Added PGPM_VERSION env variable (2.7.9) as single source of truth for pgpm version
  • Added pgpm CLI caching using actions/cache@v4
  • Pinned pgpm install to use the version from env variable

Files changed:

  • .github/workflows/ci.yml
  • .github/workflows/integration-test.yml

Review & Testing Checklist for Human

  • Verify CI passes - Confirm all test jobs complete successfully with the new configuration
  • Check PostgreSQL connectivity - The PGHOST: localhost change is critical for connecting to the pg_db service container from the runner. Verify database operations work correctly.
  • Verify pgpm version 2.7.9 compatibility - Ensure this version works with the existing test suite and test-packages --full-cycle command

Notes

This is part of a broader simplification effort across multiple repos (pgpm-modules, supabase-test-suite, pgpm-boilerplates).

Link to Devin run: https://app.devin.ai/sessions/d84286973f0a46119274e37169001af9
Requested by: Dan Lynch (@pyramation)

@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit 4f3d0b6 into main Jan 2, 2026
21 checks passed
@pyramation pyramation deleted the devin/1767337342-remove-sqitch-container branch January 2, 2026 07:20
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