Skip to content

Delete our copy of IsBuiltInRuleSet#330

Merged
tmeschter merged 1 commit intodotnet:masterfrom
tmeschter:RemoveIsBuiltInRuleSet
Feb 9, 2015
Merged

Delete our copy of IsBuiltInRuleSet#330
tmeschter merged 1 commit intodotnet:masterfrom
tmeschter:RemoveIsBuiltInRuleSet

Conversation

@tmeschter
Copy link
Contributor

This change deletes code we use to determine if a rule set file is one
that shipped with VS, and thus cannot be modified or deleted. It was
largely copied from code in
Microsoft.VisualStudio.CodeAnalysis.Sdk.UI.dll, and we should just use
the original instead.

This change deletes code we use to determine if a rule set file is one
that shipped with VS, and thus cannot be modified or deleted. It was
largely copied from code in
Microsoft.VisualStudio.CodeAnalysis.Sdk.UI.dll, and we should just use
the original instead.
@srivatsn
Copy link
Contributor

srivatsn commented Feb 9, 2015

👍

@tmeschter
Copy link
Contributor Author

@shyamnamboodiripad, @heejaechang, @jmarolf, @mavasani, could one of you take a look at this as well?

@shyamnamboodiripad
Copy link
Contributor

👍

tmeschter added a commit that referenced this pull request Feb 9, 2015
@tmeschter tmeschter merged commit 4c837aa into dotnet:master Feb 9, 2015
dibarbet pushed a commit to dibarbet/roslyn that referenced this pull request Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants