chore: remove pyramation/node-sqitch container, use ubuntu-latest directly #30
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
Removes the
pyramation/node-sqitch:20.12.0container from CI workflows, allowing jobs to run directly onubuntu-latest. This simplifies the CI setup as the custom container is no longer needed.Changes:
container: pyramation/node-sqitch:20.12.0directive from both workflowsPGHOSTfrompg_dbtolocalhost(required since jobs now run directly on the runner, not inside a container)PGPM_VERSIONenv variable (2.7.9) as single source of truth for pgpm versionactions/cache@v4Files changed:
.github/workflows/ci.yml.github/workflows/integration-test.ymlReview & Testing Checklist for Human
PGHOST: localhostchange is critical for connecting to thepg_dbservice container from the runner. Verify database operations work correctly.test-packages --full-cyclecommandNotes
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)