Conversation
b6b5b98 to
562688a
Compare
562688a to
daf1d3a
Compare
There was a problem hiding this comment.
Pull request overview
This PR migrates the containerized PostGraphile setup to PostGraphile v5, introducing a new TypeScript-based Graphile preset/configuration, new lint/format tooling, and a rebuilt Docker image workflow.
Changes:
- Add a PostGraphile v5
graphile.config.tspreset with JWT-to-pgSettingsmapping and environment validation. - Introduce TypeScript + ESLint (flat config) + Prettier configuration and pnpm dependency management.
- Replace the Docker build/runtime with a multi-stage Node + pnpm workflow and a custom entrypoint.
Reviewed changes
Copilot reviewed 11 out of 15 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
Dockerfile |
Replaces the base image and build pipeline with multi-stage pnpm-based build/lint/run. |
docker-entrypoint.sh |
Loads env vars from files and installs deps in non-production runs. |
package.json |
Defines pnpm-managed deps/devDeps for PostGraphile v5 + tooling. |
pnpm-lock.yaml |
Locks all dependencies for the new pnpm-managed stack. |
src/graphile.config.ts |
Adds PostGraphile v5 preset: services, JWT verification, and schema settings. |
src/environment.ts |
Adds helper to require/validate environment variables. |
tsconfig.json |
Adds strict TS compiler configuration for the new TS code. |
eslint.config.ts |
Adds ESLint flat config for JS/TS/JSON plus Prettier integration. |
.prettierrc.json |
Configures Prettier formatting defaults. |
.gitignore |
Ignores node_modules and ESLint cache. |
.dockerignore |
Excludes node_modules and ESLint cache from Docker build context. |
.vscode/settings.json |
Sets default formatter and ShellCheck args for local dev. |
.vscode/extensions.json |
Recommends ESLint/Prettier/ShellCheck extensions. |
.renovaterc.json |
Adjusts Renovate config (removes explicit semanticCommitType override). |
graphile-postgis-v0.2.0.tgz |
Adds vendored PostGIS plugin tarball referenced via file: dependency. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e35cf30 to
b59da32
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 16 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
91475a6 to
1ed026d
Compare
1ed026d to
892c80b
Compare
|
🎉 This PR is included in version 2.0.0-beta.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This pull request introduces a comprehensive modernization and cleanup of the project setup, focusing on containerization, development tooling, and configuration management. The changes standardize the development environment, introduce best practices for Docker and Node.js, and add robust linting and formatting workflows. Key highlights include a complete Dockerfile overhaul, new environment variable validation, and improved VS Code and linting configurations.