Cosmos icon indicating copy to clipboard operation
Cosmos copied to clipboard

added Linq (Enumerable.cs)

Open Samma2009 opened this issue 10 months ago • 5 comments

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

Samma2009 avatar Feb 07 '25 17:02 Samma2009

can you apply this patch to fix cicd

From 5ced6c3f0e2945719438aed7c70c1b1e75dc5f81 Mon Sep 17 00:00:00 2001
From: zarlo <[email protected]>
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 avatar Feb 07 '25 21:02 zarlo

can you apply this patch to fix cicd

From 5ced6c3f0e2945719438aed7c70c1b1e75dc5f81 Mon Sep 17 00:00:00 2001
From: zarlo <[email protected]>
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 done, the patch is applied

Samma2009 avatar Feb 07 '25 21:02 Samma2009

mmm it looks like cicd is still broken

zarlo avatar Feb 08 '25 00:02 zarlo

mmm it looks like cicd is still broken

Yep lol

Samma2009 avatar Feb 08 '25 07:02 Samma2009

v4 of the artifact workflows completely changes the way that they work. Due to this, you can no longer have multiple files with the same name. https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md#multiple-uploads-to-the-same-named-artifact

Lamparter avatar Feb 18 '25 09:02 Lamparter