Skip to content

Feat/enhance ci#31

Merged
randoneering merged 5 commits into
mainfrom
feat/enhance_ci
May 18, 2026
Merged

Feat/enhance ci#31
randoneering merged 5 commits into
mainfrom
feat/enhance_ci

Conversation

@randoneering
Copy link
Copy Markdown
Owner

@randoneering randoneering commented May 18, 2026

Pull Request Summary

The following PR adds a release drafter through CI and updated the CONTRIBUTING.md to reflect conventional commit messages and branch naming.

Type of Change

  • New health check
  • Bug fix
  • Performance improvement
  • Documentation update
  • Refactoring/code cleanup
  • Breaking change

Related Issues

  • Fixes #
  • Related to #
  • Closes #

Testing

PostgreSQL Version Compatibility

Has this code been tested against the following PostgreSQL versions?

  • PostgreSQL 15
  • PostgreSQL 16
  • PostgreSQL 17
  • PostgreSQL 18

Testing notes:

Managed Database Platforms

Has this code been deployed and tested on the following platforms?

  • Amazon RDS for PostgreSQL
  • Google Cloud SQL for PostgreSQL (currently unable to test)
  • Azure Database for PostgreSQL (currently unable to test)
  • Neon
  • Supabase
  • Self-managed PostgreSQL

Platform-specific notes:


Additional Notes


Security Review

  • Self-hosted runner exposure via pull_request_target (.github/workflows/release-drafter.yml): The autolabel-pr job triggers on pull_request_target from any fork PR and runs on a self-hosted runner. For a public repo this lets any external contributor execute jobs on internal infrastructure. Recommend switching the autolabeler job to ubuntu-latest.

Important Files Changed

Filename Overview
.github/workflows/release-drafter.yml New workflow using pull_request_target with self-hosted runners — security risk for a public repo; autolabeler and release-draft jobs are otherwise correctly structured
.github/release-drafter.yml New release-drafter config with full autolabeler; style label is defined but missing from categories and version-resolver patch labels, causing style PRs to be dropped from release notes
.github/CONTRIBUTING.md Adds Conventional Commits guidance and a complete branch naming table covering all commit types; minor editorial cleanup throughout

Reviews (2): Last reviewed commit: "chore(ci): reso from greptile" | Re-trigger Greptile

Greptile also left 2 inline comments on this PR.

@randoneering randoneering added documentation Improvements or additions to documentation ci labels May 18, 2026
Comment thread .github/CONTRIBUTING.md
Comment thread .github/release-drafter.yml
@randoneering
Copy link
Copy Markdown
Owner Author

@greptileai

Comment thread .github/workflows/release-drafter.yml Outdated
Comment thread .github/release-drafter.yml
@randoneering randoneering merged commit fe76bd1 into main May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant