Upgrade to .NET 7, enable WASM optimizations (#173)

* Fixed build errors caused by .NET 7 upgrade

* Updated tooling pointer

Update tooling to latest upgrade/net7

* Bump uno-check

* Skip vswinworkloads

* Update Build Script to support Pull Request Packages Again

Cleaned up some logic for diagnostic builds as well so we have a single build command

* Update tooling to use maccatalyst

* Fix for net7.0-maccatalyst, use dotnet 7.0.100

* Update tooling, additional net7-mobile error suppression

* Update tooling: More error suppression for net7 mobile

* Fixed invalid symbol references in comments

* Update editorconfig

* Update editorconfig

* Update logic for Versioning of Build and ensure to pass to Packing Script

* Use net7.0-android33.0 instead of net7.0-android

* Fix and un-suppress CS1574 errors

* Remove suppression of CS1587

* Update tooling pointer

* Fixed remaining build errors

* Update tooling to latest main

* Suppress CS1587

* Move platfiorm conditionals  to using statements

* Fix issue with XML comment for StackedNotificationsBehavior

2023-08-17T20:08:53.7233918Z      5>D:\a\Windows\Windows\components\Behaviors\src\Notification\StackedNotificationsBehavior.cs(16,70): error CS1574: XML comment has cref attribute 'InfoBar' that could not be resolved

---------

Co-authored-by: michael-hawker <24302614+michael-hawker@users.noreply.github.com>

Author: Arlodotexe

.config/dotnet-tools.json

@@ -3,7 +3,7 @@
   "isRoot": true,
   "tools": {
     "uno.check": {
-      "version": "1.10.0",
+      "version": "1.13.0",
       "commands": [
         "uno-check"
       ]

.editorconfig

@@ -434,4 +434,4 @@ csharp_style_prefer_extended_property_pattern = true:suggestion
 dotnet_diagnostic.IDE0073.severity = warning
 
 # Uno platform exposes IDisposable on Storyboard publicly when it should be internal. Ignore this.
-dotnet_code_quality.CA1001.excluded_type_names_with_derived_types = T:Windows.UI.Xaml.Media.Animation.Storyboard
+dotnet_code_quality.CA1001.excluded_type_names_with_derived_types = T:Windows.UI.Xaml.Media.Animation.Storyboard

.github/workflows/build.yml

@@ -17,8 +17,9 @@ on:
   merge_group:
 
 env:
-  DOTNET_VERSION: ${{ '7.0.x' }}
+  DOTNET_VERSION: ${{ '7.0.100' }}
   ENABLE_DIAGNOSTICS: true
+  MSBUILD_VERBOSITY: normal
   #COREHOST_TRACE: 1
   COREHOST_TRACEFILE: corehosttrace.log
   MULTI_TARGET_DIRECTORY: tooling/MultiTarget
@@ -65,6 +66,7 @@ jobs:
       # faux-ternary expression to select which platforms to build for each platform vs. duplicating step below.
       TARGET_PLATFORMS: ${{ matrix.platform != 'WinUI3' && 'all' || 'all-uwp' }}
       TEST_PLATFORM: ${{ matrix.platform != 'WinUI3' && 'UWP' || 'WinAppSdk' }}
+      VERSION_PROPERTY: ${{ github.ref == 'refs/heads/main' && format('-p:PreviewVersion=build.{0}', github.run_number) || format('-p:PreviewVersion=pull-{0}.{1}', github.event.number, github.run_number) }}
 
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:
@@ -104,7 +106,7 @@ jobs:
         run: dotnet tool restore
 
       - name: Run Uno Check to Install Dependencies
-        run: dotnet tool run uno-check --ci --fix --non-interactive --skip wsl --skip androidemulator --verbose
+        run: dotnet tool run uno-check --ci --fix --non-interactive --skip wsl --skip androidemulator --skip vswinworkloads --verbose
 
       - name: Add msbuild to PATH
         uses: microsoft/setup-msbuild@v1.3.1
@@ -122,27 +124,25 @@ jobs:
         run: powershell -version 5.1 -command "./UseUnoWinUI.ps1 3" -ErrorAction Stop
         if: ${{ matrix.platform == 'WinUI3' }}
 
-      - name: MSBuild (With diagnostics)
-        if: ${{ env.ENABLE_DIAGNOSTICS == 'true' }}
-        run: msbuild.exe CommunityToolkit.AllComponents.sln /restore /nowarn:MSB4011 -p:Configuration=Release -m /bl -v:diag
-
       - name: MSBuild
-        if: ${{ env.ENABLE_DIAGNOSTICS == 'false' }}
-        run: msbuild.exe CommunityToolkit.AllComponents.sln /restore /nowarn:MSB4011 -p:Configuration=Release -m
+        run: msbuild.exe CommunityToolkit.AllComponents.sln /restore /nowarn:MSB4011 -p:Configuration=Release -m ${{ env.VERSION_PROPERTY }} ${{ env.ENABLE_DIAGNOSTICS == 'true' && '/bl' || '' }} -v:${{ env.MSBUILD_VERBOSITY }}
 
       # Build All Packages
       - name: pack experiments
         working-directory: ./tooling/Scripts/
-        run: ./PackEachExperiment.ps1 all
+        run: ./PackEachExperiment.ps1 -extraBuildProperties "${{ env.VERSION_PROPERTY }}"
 
       # Push Packages to our DevOps Artifacts Feed (see nuget.config)
-      - name: Add source
+      - name: Add source (main)
         if: ${{ github.ref == 'refs/heads/main' }}
         run: dotnet nuget update source MainLatest --username dummy --password ${{ secrets.DEVOPS_PACKAGE_PUSH_TOKEN }}
 
+      - name: Add source (pull requests)
+        if: ${{ github.ref != 'refs/heads/main' }}
+        run: dotnet nuget add source https://pkgs.dev.azure.com/dotnet/CommunityToolkit/_packaging/CommunityToolkit-PullRequests/nuget/v3/index.json --name PullRequests --username dummy --password ${{ secrets.DEVOPS_PACKAGE_PUSH_TOKEN }}
+
       - name: Push packages
-        if: ${{ github.ref == 'refs/heads/main' }}
-        run: dotnet nuget push "**/*.nupkg" --api-key dummy --source MainLatest --skip-duplicate
+        run: dotnet nuget push "**/*.nupkg" --api-key dummy --source ${{ github.ref == 'refs/heads/main' && 'MainLatest' || 'PullRequests' }} --skip-duplicate
 
       # Run tests
       - name: Setup VSTest Path
@@ -241,7 +241,7 @@ jobs:
       # See launch.json configuration file for analogous command we're emulating here to build LINK: ../../.vscode/launch.json:CommunityToolkit.App.Wasm.csproj
       - name: dotnet build
         working-directory: ./${{ env.HEADS_DIRECTORY }}/AllComponents/Wasm/
-        run: dotnet build /r /bl
+        run: dotnet build /r /bl -v:${{ env.MSBUILD_VERBOSITY }}
 
       # TODO: Do we want to run tests here? Can we do that on linux easily?
 

components/Animations/src/ConnectedAnimations/Connected.cs

@@ -53,17 +53,17 @@ public static void SetAnchorElement(DependencyObject obj, UIElement value)
     /// <summary>
     /// Gets the connected animation key associated with the ListViewBase item being animated
     /// </summary>
-    /// <param name="obj">The <see cref="Windows.UI.Xaml.Controls.ListViewBase"/></param>
+    /// <param name="obj">The <see cref="ListViewBase"/></param>
     /// <returns>The connected animation key</returns>
     public static string GetListItemKey(DependencyObject obj)
     {
         return (string)obj.GetValue(ListItemKeyProperty);
     }
 
     /// <summary>
-    /// Sets the connected animation key for the <see cref="Windows.UI.Xaml.Controls.ListViewBase"/> item being animated
+    /// Sets the connected animation key for the <see cref="ListViewBase"/> item being animated
     /// </summary>
-    /// <param name="obj">The <see cref="Windows.UI.Xaml.Controls.ListViewBase"/></param>
+    /// <param name="obj">The <see cref="ListViewBase"/></param>
     /// <param name="value">The connected animation key</param>
     public static void SetListItemKey(DependencyObject obj, string value)
     {
@@ -73,7 +73,7 @@ public static void SetListItemKey(DependencyObject obj, string value)
     /// <summary>
     /// Gets the name of the element in the <see cref="DataTemplate"/> that is animated
     /// </summary>
-    /// <param name="obj">The <see cref="Windows.UI.Xaml.Controls.ListViewBase"/></param>
+    /// <param name="obj">The <see cref="ListViewBase"/></param>
     /// <returns>The name of the element being animated</returns>
     public static string GetListItemElementName(DependencyObject obj)
     {
@@ -83,7 +83,7 @@ public static string GetListItemElementName(DependencyObject obj)
     /// <summary>
     /// Sets the name of the element in the <see cref="DataTemplate"/> that is animated
     /// </summary>
-    /// <param name="obj">The <see cref="Windows.UI.Xaml.Controls.ListViewBase"/></param>
+    /// <param name="obj">The <see cref="ListViewBase"/></param>
     /// <param name="value">The name of the element to animate</param>
     public static void SetListItemElementName(DependencyObject obj, string value)
     {

components/Animations/src/Enums/FrameworkLayer.cs

@@ -2,6 +2,14 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+#if WINUI2
+using Composition = Windows.UI.Composition;
+using Animation = Windows.UI.Xaml.Media.Animation;
+#elif WINUI3
+using Composition = Microsoft.UI.Composition;
+using Animation = Microsoft.UI.Xaml.Media.Animation;
+#endif
+
 namespace CommunityToolkit.WinUI.Animations;
 
 /// <summary>
@@ -10,12 +18,12 @@ namespace CommunityToolkit.WinUI.Animations;
 public enum FrameworkLayer
 {
     /// <summary>
-    /// Indicates the <see cref="Windows.UI.Composition"/> APIs.
+    /// Indicates the <see cref="Composition"/> APIs.
     /// </summary>
     Composition,
-
+    
     /// <summary>
-    /// Indicates the <see cref="Windows.UI.Xaml.Media.Animation"/> APIs.
+    /// Indicates the <see cref="Animation"/> APIs.
     /// </summary>
     Xaml
 }

components/Animations/src/Enums/VisualProperty.cs

@@ -4,6 +4,12 @@
 
 namespace CommunityToolkit.WinUI.Animations;
 
+#if WINUI2
+using Windows.UI.Composition;
+#else
+using Microsoft.UI.Composition;
+#endif
+
 /// <summary>
 /// Indicates a property of a <see cref="Visual"/> object.
 /// </summary>

components/Animations/src/Expressions/ReferenceNodes/AmbientLightReferenceNode.cs

@@ -13,7 +13,7 @@ namespace CommunityToolkit.WinUI.Animations.Expressions;
 /// <summary>
 /// Class AmbientLightReferenceNode. This class cannot be inherited.
 /// </summary>
-/// <seealso cref="Microsoft.Toolkit.Uwp.UI.Animations.Expressions.ReferenceNode" />
+/// <seealso cref="ReferenceNode" />
 public sealed class AmbientLightReferenceNode : ReferenceNode
 {
     /// <summary>

components/Animations/src/Xaml/Abstract/CustomAnimation{TValue,TKeyFrame}.cs

@@ -19,6 +19,9 @@ public abstract class CustomAnimation<TValue, TKeyFrame> : ImplicitAnimation<TVa
     /// </summary>
     public string? Target { get; set; }
 
+
+// Suppression required for net7-android. The #if compilation conditionals are preventing the compiler from interpreting it correctly.
+#pragma warning disable CS1587 // XML comment is not placed on a valid language element
     /// <summary>
     /// Gets or sets the target framework layer for the animation. This is only supported
     /// for a set of animation types (see the docs for more on this). Furthermore, this is
@@ -31,6 +34,7 @@ public abstract class CustomAnimation<TValue, TKeyFrame> : ImplicitAnimation<TVa
     /// The default value is <see cref="FrameworkLayer.Xaml"/>.
     /// </summary>
     public FrameworkLayer Layer { get; set; } = FrameworkLayer.Xaml;
+#pragma warning restore CS1587 // XML comment is not placed on a valid language element
 #endif
 
     /// <inheritdoc/>

components/Animations/src/Xaml/AnimationSet.cs

@@ -5,11 +5,17 @@
 using System.Diagnostics.Contracts;
 using System.Runtime.CompilerServices;
 
+#if WINUI2
+using Windows.UI.Xaml.Media.Animation;
+#else
+using Microsoft.UI.Xaml.Media.Animation;
+#endif
+
 namespace CommunityToolkit.WinUI.Animations;
 
 /// <summary>
 /// A collection of animations that can be grouped together. This type represents a composite animation
-/// (such as <see cref="Windows.UI.Xaml.Media.Animation.Storyboard"/>) that can be executed on a given element.
+/// (such as <see cref="Storyboard"/>) that can be executed on a given element.
 /// </summary>
 public sealed class AnimationSet : DependencyObjectCollection
 {

components/Behaviors/src/Notification/StackedNotificationsBehavior.cs

@@ -13,7 +13,7 @@
 namespace CommunityToolkit.WinUI.Behaviors;
 
 /// <summary>
-/// A behavior to add the stacked notification support to <see cref="InfoBar"/>.
+/// A behavior to add the stacked notification support to <see cref="MUXC.InfoBar"/>.
 /// </summary>
 public class StackedNotificationsBehavior : BehaviorBase<MUXC.InfoBar>
 {