Conversation
There is sometimes an issue with the clone/create PR step. When that happens, the stage isn't requeuable because the VSTS upload step fails when its run again because the payload is already there. This update allows the step to clone the VS repo and create the PR to be rerun due to issues independent of the Upload step.
Also remove a task that was duplicated (I think) unnecessarily.
Split the VS PR stage separately from the upload steps
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update Dockerfile and global.json updates to v10.0.301 * Bump global.json too --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Andrew Arnott <[email protected]>
Use explicit UTF-8 appends for GitHub Actions environment files so Linux runners don't hit an invalid empty-name env entry after build. This updates both the general env propagation helper and the pipeline variable definition script. Co-authored-by: 🤖 Copilot <[email protected]> Co-authored-by: Copilot <[email protected]>
Improve and share github actions env var setting function
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Migrate coverage uploads to the supported Codecov CLI, verify its signed download, and propagate uploader failures to CI. Co-authored-by: Copilot App <[email protected]>
Fix Codecov CLI publishing
Adds a -NoCoverage switch to dotnet-test-cloud.ps1 for both the MTP and VSTest code paths. Code coverage instrumentation can destabilize a test run, and when it does there is currently no way for a repo to keep running tests on the affected leg without hand-editing this script. Also collect a crash report alongside the crash dump, and restrict the dump to the heap. On Linux the native crash report is often the only way to identify the faulting thread and instruction when a test host dies of an access violation, and a heap dump is considerably smaller than the default full dump while still containing everything needed to analyze managed state. Finally, stop discarding crash dumps that have no TRX attachment copy. The previous filter kept a .dmp only when it appeared under a TRX 'In' directory, so on GitHub Actions -- which does not request a TRX report -- every crash dump was dropped from the uploaded artifacts, making test host crashes impossible to diagnose. The dumps are now de-duplicated by file name instead, which still avoids uploading the same dump twice on Azure Pipelines. Co-authored-by: Copilot <[email protected]>
Add -NoCoverage switch and improve crash diagnostics
* Correct MTP -- forwarding with portable solution discovery. * Increase effective hang timeout to five minutes. * Batch all Codecov reports into one CLI invocation. * Preserve no-op behavior when no coverage exists.
Use ps1 splatting and exit instead of return
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…0 (#553) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This reverts commit 663bdc9. We can't update to this for another several days.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
… (557) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This latest update keeps compliant with the latest security push.
This reverts commit a853c38.
Resolve template conflicts by adopting DefaultDeny network isolation, preserving source-build analyzer conditions and repo-specific insertion reviewers, and retaining Azure Pipelines configuration. Co-authored-by: Copilot App <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the repository from Library.Template commit
1b194f80e0f543a59db1ba134b82ccb587c7a111.This merge adopts the template's tooling, SDK/package updates, skill relocation, and
DefaultDeny,CFSClean3network isolation settings. Repo-specific Azure Pipelines settings and insertion reviewers were preserved, and the deleted GitHub Actions documentation workflows remain deleted.Validation:
dotnet restore,dotnet build, and 164 net8.0 tests passed. The aggregate cloud test script also invokes net472, which reports zero tests and exits 5; net8.0 passed independently.Please merge this pull request; do not squash it.