Skip to content

Update RemoveCommand.cs #17848

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 24, 2020
Merged

Update RemoveCommand.cs #17848

merged 1 commit into from
Jan 24, 2020

Conversation

doggy8088
Copy link
Contributor

Retarget the PR #17825 to master branch.

Summary of the changes (Less than 80 chars)

  • There is a tiny typo in this file. That will produce wrong English on the console.

image

@Pilchie Pilchie added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Dec 13, 2019
@pranavkm pranavkm added this to the 5.0.0-preview1 milestone Dec 13, 2019
Copy link
Contributor

@pranavkm pranavkm left a comment

Choose a reason for hiding this comment

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

Looks good to me. @ryanbrandenburg are you aware of any external dependencies (IDEs or SDKs) that are affected by this?

@mkArtakMSFT
Copy link
Contributor

@ryanbrandenburg assigning this to you to follow up with the VS tooling team and make sure the right things will happen.

@ryanbrandenburg
Copy link
Contributor

@pranavkm I'm aware of nothing that depends on this tool.

@pranavkm pranavkm merged commit 28b0e67 into dotnet:master Jan 24, 2020
@pranavkm
Copy link
Contributor

Thanks @doggy8088

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants