Skip to content

[AOT] Remove AOT warning suppressions - #46015

Merged
JamesNK merged 1 commit into
dotnet:mainfrom
amcasey:Cleanup79425
Jan 17, 2023
Merged

JamesNK merged 1 commit into
dotnet:mainfrom
amcasey:Cleanup79425

Conversation

@amcasey

@amcasey amcasey commented Jan 11, 2023

Copy link
Copy Markdown
Member

...introduced in #45604 now that dotnet/runtime#79425 is available.

@ghost ghost added the area-runtime label Jan 11, 2023
@JamesNK

JamesNK commented Jan 11, 2023

Copy link
Copy Markdown
Member

Some more:

@amcasey

amcasey commented Jan 11, 2023

Copy link
Copy Markdown
Member Author

Oops, forgot I had a solution filter open.

@amcasey amcasey changed the title Remove AOT warning suppressions [AOT] Remove AOT warning suppressions Jan 11, 2023
@amcasey

amcasey commented Jan 12, 2023

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 3 pipeline(s).

...introduced in dotnet#45604 now that dotnet/runtime#79425 is available.
@amcasey

amcasey commented Jan 17, 2023

Copy link
Copy Markdown
Member Author

Force push is just a rebase onto main to pick up the dotnet/runtime changes.

@amcasey
amcasey marked this pull request as ready for review January 17, 2023 18:16

@eerhardt eerhardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks!

@JamesNK
JamesNK merged commit 6cade1b into dotnet:main Jan 17, 2023
@ghost ghost added this to the 8.0-preview1 milestone Jan 17, 2023
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants