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

CI update - .NET 9 GA #10678

Closed
wants to merge 6 commits into from
Closed

CI update - .NET 9 GA #10678

wants to merge 6 commits into from

Conversation

gewarren
Copy link
Contributor

@gewarren gewarren commented Nov 13, 2024

View by commit. The last commit is the documentation comments.

Copy link
Contributor

@gewarren - This PR edits one or more files whose 'source of truth' for documentation is not in this repo. Please make documentation updates in the /// comments in the dotnet/runtime repo (or dotnet/extensions repo) instead.

@gewarren gewarren added area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable. and removed area-Infrastructure labels Nov 13, 2024

This comment was marked as outdated.

 Build.Reason:Manual by Genevieve Warren
 Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=476746&view=results
 source_repo.branch:net9-ga
 source_repo.url:https://apidrop.visualstudio.com/_git/binaries
Copy link

Learn Build status updates of commit 0453d40:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/Microsoft.Extensions.DependencyInjection/AsyncStateExtensions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.DependencyInjection/OptionsServiceCollectionExtensions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.DependencyInjection/ResilienceHttpClientBuilderExtensions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.DependencyInjection/ServiceProviderOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.FileProviders/PhysicalFileProvider.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Hosting.Internal/ConsoleLifetime.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Hosting/IHost.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HedgingEndpointOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpCircuitBreakerStrategyOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpClientHedgingResiliencePredicates.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpClientResiliencePredicates.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpHedgingStrategyOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpRateLimiterStrategyOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpRetryStrategyOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpStandardResilienceOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/HttpTimeoutStrategyOptions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http.Resilience/ResilienceHandler.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Http/IHttpMessageHandlerBuilderFilter.xml ⚠️Warning View Details
[xml/Microsoft.Extensions.Options/OptionsBuilder1.xml](https://github.com/gewarren/dotnet-api-docs/blob/net9-ga-chunked/xml/Microsoft.Extensions.Options/OptionsBuilder1.xml) ⚠️Warning View Details
xml/System.Formats.Nrbf/ArrayRecord.xml ⚠️Warning View Details
xml/System.Numerics.Tensors/Tensor.xml ⚠️Warning View Details
xml/System.Numerics.Tensors/TensorPrimitives.xml ⚠️Warning View Details
xml/Microsoft.Extensions.AI/AdditionalPropertiesDictionary.xml 💡Suggestion View Details
xml/Microsoft.Extensions.AI/ConfigureOptionsChatClient.xml 💡Suggestion View Details
xml/Microsoft.Extensions.AI/ConfigureOptionsEmbeddingGenerator`2.xml 💡Suggestion View Details

This comment lists only the first 25 files in the pull request.

xml/Microsoft.Extensions.DependencyInjection/AsyncStateExtensions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Microsoft.Extensions.AsyncState.IAsyncLocalContext`1'.

xml/Microsoft.Extensions.DependencyInjection/OptionsServiceCollectionExtensions.xml

  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

xml/Microsoft.Extensions.DependencyInjection/ResilienceHttpClientBuilderExtensions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.ResiliencePipelineBuilder`1'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.ResiliencePipelineBuilder`1'.

xml/Microsoft.Extensions.DependencyInjection/ServiceProviderOptions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'BuildServiceProvider(IServiceCollection,ServiceProviderOptions)'.

xml/Microsoft.Extensions.FileProviders/PhysicalFileProvider.xml

  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 0, Column 0: [Warning: disallowed-html-tag - See documentation] HTML tag 'seealso' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

xml/Microsoft.Extensions.Hosting.Internal/ConsoleLifetime.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Microsoft.Extensions.Hosting.Internal.ConsoleLifetime.HostOptions'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Microsoft.Extensions.Hosting.Internal.ConsoleLifetime.HostOptions'.

xml/Microsoft.Extensions.Hosting/IHost.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'HostBuilder.ConfigureServices(Action<HostBuilderContext,IServiceCollection>)'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'IHostApplicationLifetime.StopApplication()'.

xml/Microsoft.Extensions.Http.Resilience/HedgingEndpointOptions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.Timeout.TimeoutStrategyOptions.Timeout'.

xml/Microsoft.Extensions.Http.Resilience/HttpCircuitBreakerStrategyOptions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.CircuitBreaker.CircuitBreakerStrategyOptions`1'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.CircuitBreaker.CircuitBreakerStrategyOptions`1'.

xml/Microsoft.Extensions.Http.Resilience/HttpClientHedgingResiliencePredicates.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.Outcome`1'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.Outcome`1'.

xml/Microsoft.Extensions.Http.Resilience/HttpClientResiliencePredicates.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.Outcome`1'.
  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.Outcome`1'.

xml/Microsoft.Extensions.Http.Resilience/HttpHedgingStrategyOptions.xml

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'Polly.CircuitBreaker.BrokenCircuitException'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@@ -78,7 +82,9 @@
<ReturnType>System.Boolean</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>
Gets a value indicating whether to generate schemas with the additionalProperties set to false for .NET objects.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What does additionalProperties refer to - could we make that a cref? @joperezr

@@ -59,7 +59,7 @@
</Attribute>
</Attributes>
<Docs>
<summary>To be added.</summary>
<summary>Provides a dictionary used as the AdditionalProperties dictionary on Microsoft.Extensions.AI objects.</summary>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same comment here @joperezr

@gewarren
Copy link
Contributor Author

Closing as cherry-picking messed up the commits. Will rerun CI and reopen.

@gewarren gewarren closed this Nov 13, 2024
@gewarren gewarren deleted the net9-ga-chunked branch November 13, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant