Skip to content

[Static Web Assets] Reworks the protocol for getting referenced assets and brings back backwards compatibility #19482

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.

Sign up for GitHub

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

Merged
javiercn merged 21 commits into from
Aug 7, 2021
Merged

[Static Web Assets] Reworks the protocol for getting referenced assets and brings back backwards compatibility #19482

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
54ccb4b
Updated baselines
javiercn Aug 4, 2021
0497c94
Test fixes
javiercn Aug 4, 2021
b47f9a9
Rework referenced projects protocol
javiercn Aug 4, 2021
7744210
Add back compat changes
javiercn Aug 4, 2021
3260200
Add back-compat tests
javiercn Aug 4, 2021
3030ae4
Fix solution
javiercn Aug 4, 2021
32099dd
Fix templatizer to account for old blazor desktop compat
javiercn Aug 4, 2021
6596e39
Feedback
javiercn Aug 6, 2021
01a33e7
Fully qualify AssetTraitName and AssetTraitValue
javiercn Aug 6, 2021
7d551c5
Templatize baselines further
javiercn Aug 6, 2021
29bb77f
Create manifest base path if it doesn't exist
javiercn Aug 7, 2021
7b7169d
Cleanup old manifest formats and only issue the developer manifest if…
javiercn Aug 7, 2021
de73467
Remove old manifest format
javiercn Aug 7, 2021
735740c
Update tests
javiercn Aug 7, 2021
d6f5c18
Fix incrementalism
javiercn Aug 7, 2021
8a3a5f6
Fix tests
javiercn Aug 7, 2021
ddc752c
Fix test
javiercn Aug 7, 2021
5b6852f
Fix baseline
javiercn Aug 7, 2021
fbbc593
Minimize json dev manifest
javiercn Aug 7, 2021
3c54b7e
Fix tests casing
javiercn Aug 7, 2021
1dd270d
Fix casing
javiercn Aug 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Cleanup old manifest formats and only issue the developer manifest if… 
… there are assets present
  • Loading branch information
@javiercn
javiercn committed Aug 7, 2021
commit 7b7169da7b00a9d32a4ff6bd997db93b23574fb9
12 changes: 3 additions & 9 deletions src/RazorSdk/Targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ Copyright (c) .NET Foundation. All rights reserved.
<PropertyGroup>

<!-- For preview only, to ease in the new implementation -->
<_GenerateLegacyManifestFormat>true</_GenerateLegacyManifestFormat>
<_GenerateLegacyManifestFormat>false</_GenerateLegacyManifestFormat>

<PrepareForRunDependsOn>StaticWebAssetsPrepareForRun;$(PrepareForRunDependsOn)</PrepareForRunDependsOn>

Expand Down Expand Up @@ -399,13 +399,7 @@ Copyright (c) .NET Foundation. All rights reserved.

<Target Name="AddStaticWebAssetsManifest" DependsOnTargets="ResolveStaticWebAssetsConfiguration">
<ItemGroup>
<ContentWithTargetPath
Include="$(StaticWebassetBuildManifestPath)"
TargetPath="$(TargetName).staticwebassets.json"
CopyToOutputDirectory="PreserveNewest"
CopyToPublishDirectory="Never" />

<ContentWithTargetPath
<ContentWithTargetPath Condition="@(StaticWebAsset) != ''"
Include="$(StaticWebAssetDevelopmentManifestPath)"
TargetPath="$(TargetName).staticwebassets.runtime.json"
CopyToOutputDirectory="PreserveNewest"
Expand Down Expand Up @@ -446,7 +440,7 @@ Copyright (c) .NET Foundation. All rights reserved.
Assets="@(StaticWebAsset)"
Source="$(PackageId)"
ManifestPath="$(StaticWebAssetDevelopmentManifestPath)">
</GenerateStaticWebAssetsDevelopmentManifest>
</GenerateStaticWebAssetsDevelopmentManifest>

<ItemGroup Condition="'$(_GenerateLegacyManifestFormat)' == 'true'">
<_LegacyContentRootDefinition Include="@(StaticWebAsset)" Condition="'%(SourceType)' != 'Discovered'"></_LegacyContentRootDefinition>
Expand Down