Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 1, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original description:

Create the command suggest in this issue @dsccommunity/SqlServerDsc/issues/2111 . The command should throw an terminating error if the specified SQL Server principal does not exist as a SQL server login. The command should use SQL Server SMO. The command should not use T-SQL to perform the task.
Also, here you can see an example how to get value from pipeline should work (parameter-block): @dsccommunity/SqlServerDsc/files/source/Public/Get-SqlDscAudit.ps1 and here you can find an example on how to thrown a terminating error: @dsccommunity/SqlServerDsc/files/source/Public/New-SqlDscAudit.ps1

Write unit tests that have at least 80% coverage, and simple integration tests for testing the command. Use the same pattern as existing commands are tested.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.


This change is Reviewable

Copilot AI requested a review from johlju August 1, 2025 12:49
@johlju johlju closed this Aug 1, 2025
@johlju johlju deleted the copilot/fix-d8f93bed-ceb6-47b8-929c-f10f274c1b0a branch August 1, 2025 12:51
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.

2 participants