Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes to readme, additional md files/diagram #368

Merged
merged 8 commits into from
Aug 25, 2020
Merged

Changes to readme, additional md files/diagram #368

merged 8 commits into from
Aug 25, 2020

Conversation

angelazhangmsft
Copy link
Contributor

No description provided.

angelazhangmsft and others added 4 commits August 14, 2020 17:36
Diagram and md files
Added diagram, changes to readme structure
CONTRIBUTING.md Outdated

**Please include as much information as possible in your issue**. The more information you provide, the more likely your issue/ask will be understood and implemented. Helpful information includes:

* What device you're running (inc. CPU type, memory, disk, etc.)
Copy link
Member

Choose a reason for hiding this comment

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

This might be useful in some cases, but I feel like the others are more important and maybe we should re-order it or add a caveat to this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reordered these and included cswinrt, SDK, .NET version

CONTRIBUTING.md Outdated Show resolved Hide resolved
@manodasanW
Copy link
Member

Diagram_CreateProjection.jpg - we build both .net 5 version and .net standard 2.0 version of the projection.

README.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@Scottj1s
Copy link
Member

Regarding the diagrams:
why is NETprojection italicized?
remove the netstandard2 comments in the diagrams? its confusing, since net5 is also supported

README.md Outdated
* NuGet package: http://aka.ms/cswinrt/nuget

C#/WinRT provides Windows Runtime (WinRT) projection support for the C# language. A "projection" is an adapter that enables programming the WinRT APIs in a natural and familiar way for the target language. For example, the C#/WinRT projection hides the details of interop between C# and WinRT interfaces, and provides mappings of many WinRT types to appropriate .NET equivalents, such as strings, URIs, common value types, and generic collections.
C#/WinRT provides Windows Runtime (WinRT) projection support for the C# language. A "projection" is an adapter that enables programming the WinRT APIs in a natural and familiar way for the target language. The C#/WinRT projection hides the details of interop between C# and WinRT interfaces, and provides mappings of many WinRT types to appropriate .NET equivalents, such as strings, URIs, common value types, and generic collections.

WinRT APIs are defined in `*.winmd` format, and C#/WinRT includes tooling that generates .NET Standard 2.0 C# code that can be compiled into interop assemblies, similar to how [C++/WinRT](https://github.com/Microsoft/cppwinrt) generates headers for the C++ language projection. This means that neither the C# compiler nor the .NET Runtime require built-in knowledge of WinRT any longer.
Copy link
Contributor

Choose a reason for hiding this comment

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

remove ".NET Standard 2.0" as that is no longer true

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Scottj1s Scottj1s merged commit bb87934 into microsoft:master Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants