-
Notifications
You must be signed in to change notification settings - Fork 559
added Linq (Enumerable.cs) #3144
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.
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
base: master
Are you sure you want to change the base?
Conversation
can you apply this patch to fix cicd From 5ced6c3f0e2945719438aed7c70c1b1e75dc5f81 Mon Sep 17 00:00:00 2001
From: zarlo <5899@zarlo.dev>
Date: Fri, 7 Feb 2025 21:06:59 +0000
Subject: [PATCH] fix: update cicd
---
.github/workflows/pr.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
index d50a098ab..da4080b09 100644
--- a/.github/workflows/pr.yml
+++ b/.github/workflows/pr.yml
@@ -159,7 +159,7 @@ jobs:
dotnet test "Cosmos\Tests\Cosmos.TestRunner.UnitTest\Cosmos.TestRunner.UnitTest.csproj" --logger "trx;LogFileName=$($env:USER_KIT_PATH)TestResults\${{ matrix.kernel }}-TestResult.trx" --filter "FullyQualifiedName~${{ matrix.kernel }}"
- name: Upload Test Logs
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: test-logs
path: ${{ env.USER_KIT_PATH }}TestResults
--
2.47.1
|
@zarlo the patch is applied
@zarlo done, the patch is applied |
mmm it looks like cicd is still broken |
Yep lol |
v4 of the artifact workflows completely changes the way that they work. |
Added Linq, direct port from https://github.com/microsoft/referencesource/blob/master/System.Core/System/Linq/Enumerable.cs
it's a pull request out of time cus of gen3 but atleast it's something new and that cosmos needed