Repository navigation
Some final touches on mege script feature - needs smoke testing - #102
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR makes final refinements to the merge script feature for generating table data scripts. The changes update UI text for clarity, integrate the generated scripts with the project's post-deployment process, and enhance the script output with helpful documentation comments.
- Updated button text from "Script Seed Data" to "Script Table Data" for better clarity
- Added automatic integration of generated scripts into the project's post-deployment configuration
- Enhanced generated merge scripts with preamble documentation and command text comments
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| src/Vsix/VSCommandTable.vsct | Updated button text to "Script Table Data" for improved clarity |
| src/Vsix/Handlers/SeedHandler.cs | Added logic to automatically append generated scripts to postdeploy.sql file |
| src/DacFXToolLib/MergeGenerator.cs | Enhanced generated scripts with documentation preamble and command text; added null check |
| README.md | Added documentation for the new Script Table Data feature |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot Create a PR to add Copilot instructions |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
related to #93