feat: remove migration archive functionality#705
Conversation
Remove the migration archive package and associated file path methods that were used by the `cld migrations archive` command. This functionality is no longer needed as migrations have been fully replaced by durable pipelines. Deleted: - engine/cld/changeset/archive/ package - EnvDir.MigrationsFilePath() method - EnvDir.MigrationsArchiveFilePath() method - MigrationsFileName constant - MigrationsArchiveFileName constant - Associated tests
🦋 Changeset detectedLatest commit: 91e29ea The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
👋 bytesizedroll, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
This PR removes the migration archive functionality that is no longer needed since migrations have been fully replaced by durable pipelines. The changes eliminate the archive package and associated file path methods from the codebase.
Changes:
- Removed the
archivepackage containing migration archival logic - Deleted file path helper methods and constants for migration files
- Removed corresponding test files
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| engine/cld/domain/paths.go | Removed constants for migration file names |
| engine/cld/domain/envdir.go | Removed methods for getting migration file paths |
| engine/cld/domain/envdir_test.go | Removed tests for deleted migration file path methods |
| engine/cld/changeset/archive/archive.go | Deleted entire archive implementation |
| engine/cld/changeset/archive/archive_test.go | Deleted archive functionality tests |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
2 participants
Remove the migration archive package and associated file path methods that were used by the
cld migrations archivecommand. This functionality is no longer needed as migrations have been fully replaced by durable pipelines.Deleted: