Conversation
| @@ -13,15 +13,15 @@ Contains files that assist with publishing to Maestro. | |||
|
|
|||
| ## [`nuget`](../nuget) | |||
There was a problem hiding this comment.
I feel like this whole document is unnecessary and should just be removed. The copilot instruction docs have the same info and much more in detail, and we also have a skill to auto-update them whenever needed.
There was a problem hiding this comment.
This doc doesn't need to be in detail but I think could be useful for someone looking at the repo for the first time and I think we should keep both docs for its own purpose rather than having folks read the other.
| @@ -2,56 +2,146 @@ | |||
|
|
|||
| The [C#/WinRT NuGet package](https://www.nuget.org/packages/Microsoft.Windows.CsWinRT/) provides tooling for the following scenarios: | |||
There was a problem hiding this comment.
A bunch of stuff in this doc is also duplicate with what's in the copilot instruction and in the interop generator skill. I think we should keep this strictly just for docs for people on how to actually use the package and generate a projection, and skip repeating how the various tools work. We could just add a link to those two copilot docs instead and say to go read those if more info on the technical details is needed.
There was a problem hiding this comment.
I agree we should keep this doc to what consumers will run into. I did end up including WinRT.Projection and WinRT.Interop and bit of those details as folks will see it in their bin folder and may wonder what it is and if they should be distributing it in the case of published libraries and unpackaged apps. And also because folks are potentially going to get errors from those tools and we want them to be able know what they are reporting issues on.
…ft/CsWinRT into manodasanw/docsUpdate3.0
2 participants
No description provided.