[in-proc backport] Use dotnet common artifacts output (#9858) #10079
Conversation
* Use dotnet common artifacts output * Fix test paths * Fix more tests * Compress-Archive fix attempt * Fix more tests * Remove verbose log from build script * Update build-extensions.ps1 * Remove release_* from esrp signing * Force build * Allow for packing via --pack * Use powershell core --pack * Force build --pack * Fix pwsh --pack * Upgrade back to .NET8 * Fix copy-paste error * Fix Verify_DepsJsonChanges * Undo SDK upgrade # Conflicts: # build/build-extensions.ps1 # test/WebJobs.Script.Tests.Integration/Controllers/ControllerScenarioTestFixture.cs # test/WebJobs.Script.Tests/DependencyTests.cs
|
@jviau Can you remove the |
This functionality is only available in .NET8 SDK. We will also need this when mult-targeting net8 & net6 anyways. I won't be checking this in until we resolve the docker build issue. |
Then I think this should be a separate PR, I think the backport PR should be purely backport only without any extra changes. We also want to put all .NET 8 changes into a feature branch first before bringing it all into in-proc. If it makes more sense, maybe these two PRs can point at the in-proc .net 8 feature branch Edit: created a feature branch: https://github.com/Azure/azure-functions-host/tree/feature/in-proc8 |
|
If this is blocked by the docker issues, we need to consider bringing this change to the feature branch used for multi-targeting. |
|
This was ported in another PR |
Backport of #9858