Skip to content

Conversation

Copy link

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 introduces a new stack config migrate command to transition user configuration files from v1 to v2 format.

  • Refactors the existing stack config command into a branch with open and new migrate subcommands
  • Updates StackConfig.Save to create parent directories and centralize backup file naming
  • Adds handler, settings, and tests for the migration command, plus README updates

Reviewed Changes

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

Show a summary per file
File Description
src/Stack/Program.cs Replaces single config command with config open and config migrate subcommands
src/Stack/Help/CommandNames.cs Adds Migrate constant for the new subcommand
src/Stack/Config/StackConfig.cs Ensures config directory exists and introduces GetV1ConfigBackupFilePath()
src/Stack/Commands/Config/MigrateConfigCommand.cs Implements the CLI command and handler for config migration
src/Stack.Tests/Config/FileStackConfigTests.cs Updates tests to use the new backup file path helper
src/Stack.Tests/Commands/Config/MigrateConfigCommandTests.cs Adds basic migration test
README.md Documents the new config open and config migrate usage

Base automatically changed from tree-structure to main June 10, 2025 21:57
@geofflamrock geofflamrock force-pushed the tree-structure-migrate branch from fc1dffc to 9dcde11 Compare June 10, 2025 22:07
@geofflamrock geofflamrock marked this pull request as ready for review June 10, 2025 22:09
@geofflamrock geofflamrock added the minor Increment the minor version when merged label Jun 10, 2025
@geofflamrock geofflamrock merged commit 6699295 into main Jun 10, 2025
18 checks passed
@geofflamrock geofflamrock deleted the tree-structure-migrate branch June 10, 2025 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Increment the minor version when merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants