* Enablee Public API analyzer
The public API analyzer was not running for all intended projects due to
a global disable. Removing that, fixing the immediate violations. Using
this PR to track anymore that may have been missed
* Update public API
---------
Co-authored-by: David Wengier <git@wengier.com>
* Make cibuild.sh consistent with CIBuild.cmd
The razor repo was inconsistent with how it handled CI build files. On Windows there was a custom build file, `eng/CBuild.cmd`, which implicitly passed along `-ci` only and required all other args to be explicity. On Unix we used `eng/common/cibuild.sh` which implicitly passed along `--restore --build --test --pack --publish --ci`.
This makes our YML file much harder to read and maintain becuase it has very different arguments across the platforms. This change addresses this by adding `eng/cibuild.sh` which effectively mirrors `eng/CIBuild.cmd` updates the YML file accordingly.
* Move to single build step
Instead of having two build steps that were `{{ if ... }`` on official builds have a single build step. This matches our behavior on Windows based builds.
* Break into restore, bulid and test phases
This matches our setup in Windows
* Use -nobl option
The --excludeCIBinaryLog option does not work on Unix due to a casing
bug. Using -nobl instead across all OS
https://github.com/dotnet/arcade/pull/12998
* fixup
* Revert the breakup into three steps
* Must restore
* Use ASP.NET Core SDK in SourceGenerator tests
* Render and verify Razor Page HTML
* Remove ASP.NET Core analyzers
* Include only DLLs from ASP.NET Core, not SDK
* Use `AssemblyLoadContext`
* Verify HTML output of view component
* Reference version 7.0.4 of ASP.NET Core DLLs
* Use empty strings as default parameter values
Co-authored-by: Fred Silberberg <fred@silberberg.xyz>
---------
Co-authored-by: Fred Silberberg <fred@silberberg.xyz>
* Add net8.0 TFM
* Fix publishing of rzls
* Reorder TFMs
* Use plural for TFM variables
* Use single TFM for rzls
* Move modern TFMs to front
* Keep major version at 7
* Add singular `$(DefaultNetCoreTargetFramework)`
* Keep extension at net7.0
* Multi-target rzls
* Publish rzls with one TFM only
This avoids an issue where the source build intermediate, built on Linux, didn't have the nupkgs in it because they were only packaged on Windows. Instead of restrictig packaging to only Windows, only push nupkgs on Windows, or in source-build situations. This is a little simpler, and has the added advantage that a developer can work on Linux or Mac and produce the nupkgs for testing or dev work.
With the introduction of Microsoft.AspNetCore.Razor.Utilities.Shared,
this assembly is no longer useful. The only types left in it were
in support telemetry, and those can easily be moved to
Microsoft.CodeAnalysis.Razor.Workspaces, which is already referenced by
most tooling projects. This will help reduce image loads in Visual
Studio.
Allison Chou
dotnet-maestro[bot]
Jared Parsons
David Wengier
Jan Jones
Maryam Ariyan
Matt Mitchell
Nikola Milosavljevic
Dustin Campbell
Rikki Gibson
Jared Parsons