Feat/cliupd#237
Merged
pyramation merged 7 commits intomainfrom Sep 17, 2025
Merged
Conversation
- Implement lql add <changeName> command - Support --requires and --note options - Create deploy/revert/verify SQL file triplets - Update plan files with new changes - Handle nested paths with mkdir -p behavior - Follow existing CLI command patterns Co-Authored-By: Dan Lynch <pyramation@gmail.com>
- Remove complex package selection logic from CLI - Move all validation into addChange() method - Follow analyze/rename pattern for simplicity - Remove excessive logging and directory changing - Use direct argv.cwd pattern Co-Authored-By: Dan Lynch <pyramation@gmail.com>
- Add command is text/SQL only operation as requested - No database connection needed for file creation Co-Authored-By: Dan Lynch <pyramation@gmail.com>
- CLI tests cover command execution, input prompting, file creation - Core tests cover addChange method, validation, error handling - Test nested paths, dependencies, and SQL file content - Verify no transaction statements in generated SQL files - Follow established testing patterns from existing test suites Co-Authored-By: Dan Lynch <pyramation@gmail.com>
- Test 4-level deep path: schema/myschema/tables/mytable - Verify mkdir -p behavior works for deeply nested directories - Ensure SQL files are created in correct nested structure - Both CLI and core tests pass with new test case Co-Authored-By: Dan Lynch <pyramation@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.