-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[Blazor] Pins the SDK version to 3.1 and forces Blazor projects to compile against it #17479
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
Changes from all commits
aba66e7
18ad893
0cb6c6b
6dddd77
9455263
aa94b43
ac96ee4
dfa4443
853b758
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,15 +11,14 @@ | |
<Description>Development server for use when building Blazor applications.</Description> | ||
<!-- Set this to false because assemblies should not reference this assembly directly, (except for tests, of course). --> | ||
<IsProjectReferenceProvider>false</IsProjectReferenceProvider> | ||
|
||
<!-- This is so that we add the FrameworkReference to Microsoft.AspNetCore.App --> | ||
<UseLatestAspNetCoreReference>true</UseLatestAspNetCoreReference> | ||
</PropertyGroup> | ||
|
||
<ItemGroup> | ||
<Reference Include="Microsoft.AspNetCore.Blazor.Server" /> | ||
<Reference Include="Microsoft.AspNetCore.Components.Server" /> | ||
<Reference Include="Microsoft.AspNetCore.ResponseCompression" /> | ||
<Reference Include="Microsoft.AspNetCore" /> | ||
<Reference Include="Microsoft.Extensions.CommandLineUtils.Sources" /> | ||
<Reference Include="Microsoft.Extensions.Hosting" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We'll have to add all these back in There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We will likely have to resolve/revert the conflict at one point and that should be it. |
||
</ItemGroup> | ||
|
||
<!-- Pack settings --> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,5 +6,8 @@ | |
<VersionPrefix>3.1.0</VersionPrefix> | ||
<PreReleaseVersionLabel>preview4</PreReleaseVersionLabel> | ||
<DotNetFinalVersionKind /> | ||
<!-- This property points to the latest released Microsoft.AspNetCore.App version it needs to be updated to | ||
target the latest patch before a preview release. --> | ||
<LatestAspNetCoreReferenceVersion>3.1.0</LatestAspNetCoreReferenceVersion> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would it make sense to define this like the following?
... or are there cases where this might not match up? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure about it, its one of the follow-ups I intend to do with the build folks next week. |
||
</PropertyGroup> | ||
</Project> | ||
</Project> |
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this updated automatically? Is there a risk that your change here will get overwritten by automation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about this, but it will come from a PR from maestro and we can decide what to do there.