Skip to content

Delete the public TriggerDiagnosticDescriptor type. - #1065

Merged
shyamnamboodiripad merged 1 commit into
dotnet:masterfrom
shyamnamboodiripad:DeleteTriggerDiagnosticDescriptor
Mar 6, 2015
Merged

Delete the public TriggerDiagnosticDescriptor type.#1065
shyamnamboodiripad merged 1 commit into
dotnet:masterfrom
shyamnamboodiripad:DeleteTriggerDiagnosticDescriptor

Conversation

@shyamnamboodiripad

Copy link
Copy Markdown
Contributor

And seal its base type DiagnosticDescriptor.

(Fixes #789)

@shyamnamboodiripad

Copy link
Copy Markdown
Contributor Author

@tmeschter @mavasani @heejaechang @srivatsn could you please take a look?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe add a comment about the reasons for message=null and the notconfigurable tag?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure I'll add a comment for this.

@srivatsn

srivatsn commented Mar 6, 2015

Copy link
Copy Markdown
Contributor

👍

Comment thread src/Test/Utilities/DescriptorFactory.cs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Doc comments here would be helpful.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't add doc comments since this is for tests only :) But will add.

@mavasani

mavasani commented Mar 6, 2015

Copy link
Copy Markdown
Contributor

👍

1 similar comment
@heejaechang

Copy link
Copy Markdown
Contributor

👍

And seal its base type DiagnosticDescriptor.

(Fixes dotnet#789)
@shyamnamboodiripad
shyamnamboodiripad force-pushed the DeleteTriggerDiagnosticDescriptor branch from d65864e to f1045fd Compare March 6, 2015 08:41
@tmeschter

Copy link
Copy Markdown
Contributor

👍

shyamnamboodiripad added a commit that referenced this pull request Mar 6, 2015
…ticDescriptor

Delete the public TriggerDiagnosticDescriptor type.

And seal its base type DiagnosticDescriptor.

(Fixes #789, #843)
@shyamnamboodiripad
shyamnamboodiripad merged commit 3267d3c into dotnet:master Mar 6, 2015
@shyamnamboodiripad
shyamnamboodiripad deleted the DeleteTriggerDiagnosticDescriptor branch March 6, 2015 19:09
dibarbet pushed a commit that referenced this pull request Aug 18, 2026
Add LightupHelpers.CreatePropertyAccessor
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.

TriggerDiagnosticDescriptor doesn't prevent empty preview

7 participants