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

Backport https://github.com/dotnet/sdk/pull/19873 to RC1 #19963

Merged
merged 1 commit into from
Aug 24, 2021

Conversation

javiercn
Copy link
Member

@javiercn javiercn commented Aug 20, 2021

Description

Small fixes to some more advanced scenarios for static web assets

  • Fixes an issue with projects not working with reference assemblies using HintPath.
  • Fixes an issue where files without an extension where being included inside an additional folder with the file name on nuget packages.
  • Adds additional error logging for the brotli compression tool.
  • Adds additional login for the gzip compress task.
  • Adds editor config files in the blazor wasm and razor sdk folders to handle 'var' usage preferences in ASP.NET Core projects.
  • Performs a more selective filtering for static web assets candidates from the build candidates in blazor webassembly applications.
  • Fixes an issue with AppendTargetFramework=false causing issues

Customer Impact

Some existing customers aren't able to update to the latest preview due to some of the issues. Specifically:

  • Fixes an issue with projects not working with reference assemblies using HintPath.
  • Fixes an issue with AppendTargetFramework=false causing issues

Other changes are fixes to other scenarios we believe didn't work before either

Regression?

  • Yes (for the two issues outlined above)
  • No

Preview6

Risk

  • Low

We have extensive end to end test coverage (all fixes include new automation to validate the scenarios) and the changes don't affect core scenarios.

Verification

  • Automated

dotnet/aspnetcore#35349
dotnet/aspnetcore#32744
dotnet/aspnetcore#29561
dotnet/aspnetcore#35561

* Fixes an issue with projects not working with reference assemblies
  using HintPath.
* Fixes an issue where files without an extension where being included
  inside an additional folder with the file name on nuget packages.
* Adds additional error logging for the brotli compression tool.
* Adds additional login for the gzip compress task.
* Adds editor config files in the blazor wasm and razor sdk folders
  to handle 'var' usage preferences in ASP.NET Core projects.
* Performs a more selective filtering for static web assets candidates
  from the build candidates in blazor webassembly applications.
* Fixes an issue with AppendTargetFramework=false causing issues

dotnet/aspnetcore#35349
dotnet/aspnetcore#32744
dotnet/aspnetcore#29561
dotnet/aspnetcore#35561
@javiercn javiercn requested a review from pranavkm August 20, 2021 21:32
@javiercn javiercn added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch and removed Area-WebSDK labels Aug 20, 2021
@javiercn javiercn changed the title [Static web assets] Several fixes for additional scenarios Backport https://github.com/dotnet/sdk/pull/19873 to RC1 Aug 20, 2021
@Pilchie
Copy link
Member

Pilchie commented Aug 20, 2021

I'm in favor of this fix for RC1, given the VS Feedback items behind it.

@javiercn
Copy link
Member Author

@Pilchie @pranavkm good to merge?

Copy link
Contributor

@pranavkm pranavkm left a comment

Choose a reason for hiding this comment

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

Signing off since this is a straight backport. That said, this might need to go thru tactics to be merged.

@javiercn
Copy link
Member Author

@pranavkm who is coordinating that

@Pilchie
Copy link
Member

Pilchie commented Aug 24, 2021

I'll discuss it today at Tactics.

@javiercn javiercn merged commit bd8d08f into release/6.0.1xx-rc1 Aug 24, 2021
@javiercn javiercn deleted the javiercn/backport-19873 branch August 24, 2021 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch Servicing-approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants