Skip to content

Update SDK#49297

Merged
captainsafia merged 3 commits intomainfrom
safia/update-sdk-07102023
Jul 10, 2023
Merged

Update SDK#49297
captainsafia merged 3 commits intomainfrom
safia/update-sdk-07102023

Conversation

@captainsafia
Copy link
Copy Markdown
Contributor

@captainsafia captainsafia commented Jul 10, 2023

No description provided.

@captainsafia captainsafia requested review from a team and wtgodbe as code owners July 10, 2023 14:39
@ghost ghost added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework labels Jul 10, 2023
@ghost
Copy link
Copy Markdown

ghost commented Jul 10, 2023

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

@captainsafia
Copy link
Copy Markdown
Contributor Author

captainsafia commented Jul 10, 2023 •
edited
Loading

src/Http/Routing/test/UnitTests/RoutingMetricsTests.cs(144,32): error TBD: (NETCORE_ENGINEERING_TELEMETRY=Build) 'Microsoft.Extensions.Telemetry.Testing.Metering.CollectedMeasurement' is for evaluation purposes only and is subject to change or removal in future updates. (https://aka.ms/dotnet-extensions-warnings/TBD)

This error in RoutingMetricTests and others looks like a result of dotnet/extensions#4130. It seems sufficient to suppress this diagnostic given its in test. It's not clear to me how to suppress given it hasn't been designated a diagnostic ID yet...

Update: it looks like it is feasible by passing the TBD diagnostic ID.

cc: @RussKie @joperezr Do we expect a more permanent diagnostic ID to land soon for this? Also, is suppressing the diagnostic in test code is the right approach?

@captainsafia captainsafia force-pushed the safia/update-sdk-07102023 branch from d667e15 to c30eaf5 Compare July 10, 2023 15:37
wtgodbe
wtgodbe reviewed Jul 10, 2023
View reviewed changes
wtgodbe
wtgodbe requested changes Jul 10, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

Don't want to overwrite existing NoWarn

@wtgodbe
Update Directory.Build.props
29f45f2 
Co-authored-by: William Godbe <wigodbe@microsoft.com>
@captainsafia captainsafia requested a review from wtgodbe July 10, 2023 18:04
@RussKie RussKie mentioned this pull request Jul 10, 2023
Closed
@RussKie
Copy link
Copy Markdown
Contributor

RussKie commented Jul 10, 2023

src/Http/Routing/test/UnitTests/RoutingMetricsTests.cs(144,32): error TBD: (NETCORE_ENGINEERING_TELEMETRY=Build) 'Microsoft.Extensions.Telemetry.Testing.Metering.CollectedMeasurement' is for evaluation purposes only and is subject to change or removal in future updates. (aka.ms/dotnet-extensions-warnings/TBD)

This error in RoutingMetricTests and others looks like a result of dotnet/extensions#4130. It seems sufficient to suppress this diagnostic given its in test. It's not clear to me how to suppress given it hasn't been designated a diagnostic ID yet...

Update: it looks like it is feasible by passing the TBD diagnostic ID.

cc: @RussKie Igor Velikorossov FTE @joperezr Jose Perez Rodriguez FTE Do we expect a more permanent diagnostic ID to land soon for this? Also, is suppressing the diagnostic in test code is the right approach?

Yes, it should be safe to suppress. This is still WIP (dotnet/extensions#4161). Eventually, there will be IDs defined.

/cc: @geeknoid

RussKie added a commit to dotnet/extensions that referenced this pull request Jul 10, 2023
@RussKie
Update global.json
19ea375 
As per dotnet/aspnetcore#49297
@RussKie RussKie mentioned this pull request Jul 10, 2023
Merged
@captainsafia captainsafia merged commit a944ed3 into main Jul 10, 2023
@captainsafia captainsafia deleted the safia/update-sdk-07102023 branch July 10, 2023 23:10
@ghost ghost added this to the 8.0-preview7 milestone Jul 10, 2023
RussKie added a commit to dotnet/extensions that referenced this pull request Jul 11, 2023
@RussKie
Update SDK (#4162)
b7bc228 
* Update global.json

As per dotnet/aspnetcore#49297

* Update Directory.Build.targets

Suppress TBD

* Fix build
@RussKie RussKie mentioned this pull request Jul 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javiercn javiercn javiercn approved these changes

@wtgodbe wtgodbe wtgodbe approved these changes

@Tratcher Tratcher Awaiting requested review from Tratcher

@halter73 halter73 Awaiting requested review from halter73

Assignees

No one assigned

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Milestone

8.0-preview7

Development

Successfully merging this pull request may close these issues.

4 participants

@captainsafia @RussKie @javiercn @wtgodbe