Skip to content

Commit 21aace3

Browse files
committed
Adding OutputType to cmdlets per code review comments
1 parent 650669a commit 21aace3

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

src/ResourceManager/Sql/Commands.Sql/ServerCommunicationLink/Cmdlet/GetAzureSqlServerCommunicationLink.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@
1818

1919
namespace Microsoft.Azure.Commands.Sql.ServerCommunicationLink.Cmdlet
2020
{
21-
[Cmdlet(VerbsCommon.Get, "AzureRmSqlServerCommunicationLink",
22-
ConfirmImpact = ConfirmImpact.None)]
21+
[Cmdlet(VerbsCommon.Get, "AzureRmSqlServerCommunicationLink",
22+
ConfirmImpact = ConfirmImpact.None), OutputType(typeof(AzureSqlServerCommunicationLinkModel))]
2323
public class GetAzureSqlServerCommunicationLink : AzureSqlServerCommunicationLinkCmdletBase
2424
{
2525
/// <summary>

src/ResourceManager/Sql/Commands.Sql/ServerCommunicationLink/Cmdlet/NewAzureSqlServerCommunicationLink.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ namespace Microsoft.Azure.Commands.Sql.ServerCommunicationLink.Cmdlet
2727
/// Cmdlet to create a new Azure Sql server communication link
2828
/// </summary>
2929
[Cmdlet(VerbsCommon.New, "AzureRmSqlServerCommunicationLink",
30-
ConfirmImpact = ConfirmImpact.Low)]
30+
ConfirmImpact = ConfirmImpact.Low), OutputType(typeof(AzureSqlServerCommunicationLinkModel))]
3131
public class NewAzureSqlServerCommunicationLink : AzureSqlServerCommunicationLinkCmdletBase
3232
{
3333
/// <summary>

src/ResourceManager/Sql/Commands.Sql/ServerCommunicationLink/Cmdlet/RemoveAzureSqlServerCommunicationLink.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@
2121
namespace Microsoft.Azure.Commands.Sql.ServerCommunicationLink.Cmdlet
2222
{
2323
[Cmdlet(VerbsCommon.Remove, "AzureRmSqlServerCommunicationLink",
24-
SupportsShouldProcess = true,
25-
ConfirmImpact = ConfirmImpact.High)]
24+
SupportsShouldProcess = true,
25+
ConfirmImpact = ConfirmImpact.High), OutputType(typeof(AzureSqlServerCommunicationLinkModel))]
2626
public class RemoveAzureSqlServerCommunicationLink : AzureSqlServerCommunicationLinkCmdletBase
2727
{
2828
/// <summary>

src/ResourceManager/Sql/Commands.Sql/ServerCommunicationLink/Cmdlet/SetAzureSqlServerCommunicationLink.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ namespace Microsoft.Azure.Commands.Sql.ServerCommunicationLink.Cmdlet
2424
/// Cmdlet to create a new Azure Sql Database server communication link
2525
/// </summary>
2626
[Cmdlet(VerbsCommon.Set, "AzureRmSqlServerCommunicationLink",
27-
ConfirmImpact = ConfirmImpact.Medium)]
27+
ConfirmImpact = ConfirmImpact.Medium), OutputType(typeof(AzureSqlServerCommunicationLinkModel))]
2828
public class SetAzureSqlServerCommunicationLink : AzureSqlServerCommunicationLinkCmdletBase
2929
{
3030
/// <summary>

0 commit comments

Comments
 (0)