fix: avoid Liquibase include directive in explanatory headers #67

Merged

james.maes merged 1 commits from james.maes/liquibase-header-comments AGit into develop 2026-11-03 21:00:37 +00:00

Conversation 0 Commits 1 Files Changed 2

+2 -2

james.maes commented 2026-11-02 20:16:07 +00:00

Owner

Copy link

Copy source

Comment-only fix for the GG Forge consumer. The CLI interprets the explanatory Include-it header as a licensed directive. No changeset content changes. Real packaged PostgreSQL verification passes: Cluster 8 changesets and Alerts 58. Please carry through the normal supported artifact release.

Comment-only fix for the GG Forge consumer. The CLI interprets the explanatory Include-it header as a licensed directive. No changeset content changes. Real packaged PostgreSQL verification passes: Cluster 8 changesets and Alerts 58. Please carry through the normal supported artifact release.

james.maes added 1 commit 2026-11-02 20:16:08 +00:00

fix: avoid Liquibase include directive in explanatory headers ... 2954345e5d

Use ordinary prose in changelog headers so the CLI does not interpret documentation as a licensed include directive. Changeset content is unchanged. Actual packaged schema and defaults pass the PostgreSQL CLI gate (cluster 8 changesets; alerts 58).

Co-Authored-By: GPT-6 (Codex) <noreply@openai.com>

james.maes merged commit 2854e5336e into develop 2026-11-03 21:00:37 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

Architecture review: coupling, cohesion, UI API

bug

Something isn't working

dependencies

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

github_actions

Pull requests that update GitHub Actions code

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

java

Pull requests that update java code

question

Further information is requested

release:rc

Merging this develop PR cuts the next release candidate

secuurity

ux

Admin UX / information architecture

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

forge-recovery (Forge Recovery) james.maes (James Maes)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: QRun-IO/qbit-alerts#67