Skip to content

Conversation

@iSazonov
Copy link
Collaborator

PR Summary

Related #4708.

Format Microsoft.PowerShell.Commands.Management.csproj by codeformatter with default options.

The codeformatter is used that I compiled with newest Roslyn packages (version 2.8.2).

PR Checklist

} = new string[0];

// Include
// Include
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps delete this comment

public virtual string[] Exclude
{
get;
// get
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete

// get
set;
// set
// set
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

} = new string[0];

// Exclude
// Exclude
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

} = new string[0];

// Exclude
// Exclude
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete

@iSazonov
Copy link
Collaborator Author

@SteveL-MSFT You comments was addressed

Copy link
Member

@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work! Thanks @iSazonov

@daxian-dbw daxian-dbw added the Issue-Code Cleanup the issue is for cleaning up the code with no impact on functionality label Jul 25, 2018
@daxian-dbw daxian-dbw merged commit d97c9e7 into PowerShell:master Jul 26, 2018
@iSazonov iSazonov deleted the fmt-codeformatter-management branch July 27, 2018 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue-Code Cleanup the issue is for cleaning up the code with no impact on functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants