Match C# artifactId and version in DependencyInsight #120
Conversation
| example = "Microsoft*", | ||
| required = false) | ||
| @Nullable | ||
| String artifactIdPattern; |
There was a problem hiding this comment.
In .NET / NuGet land people talk about packages rather than artifacts, so we may want to rename things and also adjust display names and descriptions accordingly, as to not confuse the user.
There was a problem hiding this comment.
Would you even go as far as from DependencyInsight to PackageInsight ?
There was a problem hiding this comment.
Not sure. I think dependency might still be a generic enough term. While the C# project files use the term package references, I think developers probably also refer to them as dependencies. But I am guessing a bit here.
There was a problem hiding this comment.
Then let's stick with just the renamed field for now then, to keep symmetry with the Maven/Gradle and Nodejs recipes and visualizations we already have. We can gather feedback around this from the first users.
Should help fill out DevCenter for C#: https://app.moderne.io/devcenter/C%23