Skip to content

Conversation

@adamgauthier
Copy link
Contributor

@adamgauthier adamgauthier commented May 19, 2020

Improve mention of IDE experience scenario so that it links to it.
Acknowledge trailing JSON comma. Fix FileTransformGenerator example
indent.

Copy link
Contributor Author

@adamgauthier adamgauthier May 19, 2020

Choose a reason for hiding this comment

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

I wasn't exactly sure if I needed to re-arrange the entire paragraph here since this makes this line much longer. There seems to be inconsistent style throughout the file where some paragraphs are trimmed at ~90 characters. Either way the rendered markdown doesn't seem to be affected.

Improve mention of IDE experience scenario so that it links to it.
Acknowledge trailing JSON comma. Fix FileTransformGenerator example
indent.
@adamgauthier adamgauthier force-pushed the source-generators-cookbook-fixes branch from 4767b2a to a43802e Compare July 14, 2020 21:03
@chsienki chsienki merged commit 421bf4b into dotnet:master Jul 16, 2020
@chsienki
Copy link
Member

Thanks @louistio! Apologies for the delay in getting this merged

@ghost ghost added this to the Next milestone Jul 16, 2020
333fred added a commit to 333fred/roslyn that referenced this pull request Jul 17, 2020
* upstream/master: (86 commits)
  Better client / server logging (dotnet#46079)
  Fix OptProf config
  Update src/VisualStudio/LiveShare/Impl/AbstractGoToDefinitionWithFindUsagesServiceHandler.cs
  Do not execute RemoveUnnecessaryInlineSuppressions on generated code
  Update dependencies from https://github.com/dotnet/arcade build 20200715.6 (dotnet#46086)
  Use ISpanMapper before sending cross file results to LSP
  Add additional module initializers tests from review (dotnet#46020)
  Fix type in OptProf configuration
  bump to 500
  Fixed VS crash during implicit conversion of null object in nullable walker (dotnet#45974)
  Rename variable
  Fix KeyNotFound exception in RemoveUnnecessaryInlineSuppressionsDiagnosticAnalyzer
  Check modifiers on record positional members (dotnet#45898)
  Fix typos and link in Source Generators cookbook (dotnet#44372)
  Remove reference to non-existing VisualStudioInteractiveComponents.vsix from deployment VSIX. (dotnet#45979)
  Address feedback + fix tests
  Update dependencies from https://github.com/dotnet/roslyn build 20200711.1 (dotnet#45932)
  Make MetadataTypeName non-copyable
  Fix usage of GetService extension
  Consolidate service provider extensions
  ...
@JoeRobich JoeRobich modified the milestones: Next, 16.8.P1 Jul 20, 2020
333fred added a commit to 333fred/roslyn that referenced this pull request Jul 23, 2020
…to function-pointer-type-lookup

* upstream/features/function-pointers: (86 commits)
  Better client / server logging (dotnet#46079)
  Fix OptProf config
  Update src/VisualStudio/LiveShare/Impl/AbstractGoToDefinitionWithFindUsagesServiceHandler.cs
  Do not execute RemoveUnnecessaryInlineSuppressions on generated code
  Update dependencies from https://github.com/dotnet/arcade build 20200715.6 (dotnet#46086)
  Use ISpanMapper before sending cross file results to LSP
  Add additional module initializers tests from review (dotnet#46020)
  Fix type in OptProf configuration
  bump to 500
  Fixed VS crash during implicit conversion of null object in nullable walker (dotnet#45974)
  Rename variable
  Fix KeyNotFound exception in RemoveUnnecessaryInlineSuppressionsDiagnosticAnalyzer
  Check modifiers on record positional members (dotnet#45898)
  Fix typos and link in Source Generators cookbook (dotnet#44372)
  Remove reference to non-existing VisualStudioInteractiveComponents.vsix from deployment VSIX. (dotnet#45979)
  Address feedback + fix tests
  Update dependencies from https://github.com/dotnet/roslyn build 20200711.1 (dotnet#45932)
  Make MetadataTypeName non-copyable
  Fix usage of GetService extension
  Consolidate service provider extensions
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants