Roslyn: Naming for the new suppression scope

Created on 15 Nov 2018  路  8Comments  路  Source: dotnet/roslyn

https://github.com/dotnet/roslyn/pull/31092 added a new namespace scope to suppress diagnostics in namespace and all its descendants. The current naming is "NamespaceAndChildren", but we probably want a better name that is easier to understand.

4 - In Review Area-IDE Bug

All 8 comments

馃摑 'children' is not the correct term because the implementation uses a transitive closure. 'descendants' is the transitive closure of 'children'.

Also tagging @JosefPihrt

I like namespaceanddescendants or just namespacedescendants. Some other suggestions:

namespaceplus
namespaceandbelow
namespacerecursive

Besides my original proposal "NamespaceAndDescendants" I have another suggestion: "NamespaceAndContent"

We have decided to rename the scope to NamespaceAndDescendants. I will send a PR soon.

So, it's merged. Does that mean I can use this?
I'm on VS 15.9.4 and this doesn't seem to work.
Any idea why?

@XtroTheArctic It was merged into a branch that is in the 16.0 line. So you'll need a very recent build of VS2019. Probably VS2019 preview 2 (which hasn't been released yet).

That's awesome news! Thank you very much.

Was this page helpful?
0 / 5 - 0 ratings