Skip to content

Delete Visual Studio include files#7921

Merged
gewarren merged 16 commits intodotnet:masterfrom
gewarren:no-includes
Oct 2, 2018
Merged

Delete Visual Studio include files#7921
gewarren merged 16 commits intodotnet:masterfrom
gewarren:no-includes

Conversation

@gewarren
Copy link
Copy Markdown
Contributor

@gewarren gewarren commented Sep 26, 2018

Now that the agreed upon work to update high page view topics for older versions of VS is complete (and the low page view topics are archived), this PR completes the work to delete all the VS includes files and replace references with literal strings. Also bullets See Also lists in some places.

@gewarren gewarren changed the title Delete Visual Studio include files [WIP] Delete Visual Studio include files Sep 26, 2018
@gewarren gewarren changed the title [WIP] Delete Visual Studio include files Delete Visual Studio include files Sep 27, 2018
Copy link
Copy Markdown
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

I browsed some of the published files, verified that the build was clean (no warnings).

I'll approve this.

You can :shipit: when ready @gewarren

@gewarren
Copy link
Copy Markdown
Contributor Author

gewarren commented Oct 2, 2018

@BillWagner Thanks for reviewing! Merging it now.

@gewarren gewarren merged commit 874b918 into dotnet:master Oct 2, 2018
@gewarren gewarren deleted the no-includes branch October 21, 2019 20:05
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