Skip to content

Comments

chore: remove ir release module and update doc#118

Merged
tianzhou merged 2 commits intomainfrom
simplify
Oct 23, 2025
Merged

chore: remove ir release module and update doc#118
tianzhou merged 2 commits intomainfrom
simplify

Conversation

@tianzhou
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 23, 2025 10:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the standalone IR package release infrastructure, consolidating the IR package as an internal component of the main pgschema project rather than a separately versioned Go module.

Key changes:

  • Removed IR package release workflow and version management
  • Eliminated IR integration tests that validated SQL parsing vs database inspection equivalence
  • Updated documentation to reflect inspector-only approach

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
testdata/dump/README.md Removed documentation about IR integration tests
ir/ir_integration_test.go Deleted entire integration test suite (673 lines)
ir/go.mod Removed separate go.mod for IR package
ir/VERSION Removed IR version file
ir/README.md Updated to reflect inspector-only usage and removed parsing/versioning sections
go.mod Consolidated IR as internal package, moved dependencies to main level
CLAUDE.md Updated architecture notes to reflect IR as internal package
.github/workflows/release-ir.yml Removed IR package release workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tianzhou tianzhou merged commit 00b961b into main Oct 23, 2025
2 checks passed
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.

1 participant