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

Source Generator showing packaging and multiple roslyn versions #50

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

clairernovotny
Copy link

@clairernovotny clairernovotny commented Nov 30, 2021

Sample project for NuGet docs based on Refit

@sharwell

This will be referenced by the docs attached to https://github.com/NuGet/docs.microsoft.com-nuget/issues/2595

@clairernovotny
Copy link
Author

Looking for @sharwell to review as this is based on his work.

@clairernovotny
Copy link
Author

Note: there's an easier way to do this with .NET 6+. The current example is over complicated if we set that as a baseline.

@@ -24,6 +24,8 @@ This project has adopted the [Microsoft Open Source Code of Conduct](https://ope

1. [PackageSourceMappingExample](PackageSourceMappingExample) is a sample solution that shows how to configure Package Source Mapping for a couple projects and packages. [The README](PackageSourceMappingExample/README.md) contains many smaller example scenarios and results.

1. [SourceGeneratorSample](SourceGeneratorSample) is a sample solution showing how to build and pack a Source Generator that targets multiple Roslyn API versions. For more information see [Source Generators](https://docs.microsoft.com/en-us/dotnet/csharp/roslyn-sdk/source-generators-overview)

Choose a reason for hiding this comment

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

I was seeking examples of linking to other docs and found this.
Should "en-us" be removed, at least? Or do we just leave that and let the client redirect to the correct language?

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.

2 participants