From e681d26f03ff69401974388c9a2a67a5c4c13f2c Mon Sep 17 00:00:00 2001 From: Samman Das <105208798+RayBreeze@users.noreply.github.com> Date: Tue, 7 Oct 2025 08:26:05 +0530 Subject: [PATCH 1/5] Update CMake action to use latest version --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d42bd42..79db4d8 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -18,7 +18,7 @@ jobs: - name: Setup CMake (Ubuntu) if: matrix.os == 'ubuntu-latest' - uses: lukka/get-cmake@v3 + uses: lukka/get-cmake@latest - name: Setup MSVC dev environment (Windows) if: matrix.os == 'windows-latest' From b2d2c6a7a1a0dd250b683da64a8ba85b05106063 Mon Sep 17 00:00:00 2001 From: Samman Das <105208798+RayBreeze@users.noreply.github.com> Date: Tue, 7 Oct 2025 08:34:49 +0530 Subject: [PATCH 2/5] Simplify build step by removing cd command Removed unnecessary 'cd build' command before building. --- .github/workflows/ci.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 79db4d8..dcd0916 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -34,7 +34,6 @@ jobs: - name: Build run: | - cd build cmake --build . --config Release - name: Run tests From de76dfdc7e925b8f76dee716a1333b719c61485d Mon Sep 17 00:00:00 2001 From: Samman Das <105208798+RayBreeze@users.noreply.github.com> Date: Tue, 7 Oct 2025 08:38:32 +0530 Subject: [PATCH 3/5] Testing the Combination of configure and build steps in CI workflow Updated CI workflow to combine configure and build steps. --- .github/workflows/ci.yml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index dcd0916..d4a5f87 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -26,16 +26,14 @@ jobs: with: arch: x64 - - name: Configure + - name: Configure and Build run: | mkdir -p build cd build cmake -DCMAKE_BUILD_TYPE=Release .. - - - name: Build - run: | cmake --build . --config Release + - name: Run tests if: always() run: | From cc2744282e9dd1314bcd04250158ae087982ee04 Mon Sep 17 00:00:00 2001 From: Samman Das <105208798+RayBreeze@users.noreply.github.com> Date: Tue, 7 Oct 2025 08:56:30 +0530 Subject: [PATCH 4/5] Refactor CI workflow to install dependencies and build --- .github/workflows/ci.yml | 20 ++++++-------------- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d4a5f87..ec9dc31 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -16,23 +16,15 @@ jobs: steps: - uses: actions/checkout@v4 - - name: Setup CMake (Ubuntu) + - name: Install dependencies if: matrix.os == 'ubuntu-latest' - uses: lukka/get-cmake@latest + run: sudo apt-get install -y build-essential cmake - - name: Setup MSVC dev environment (Windows) - if: matrix.os == 'windows-latest' - uses: TheMrMilchmann/setup-msvc-dev@v4 - with: - arch: x64 - - - name: Configure and Build - run: | - mkdir -p build - cd build - cmake -DCMAKE_BUILD_TYPE=Release .. - cmake --build . --config Release + - name: Configure + run: cmake -S . -B build -DCMAKE_BUILD_TYPE=Release + - name: Build + run: cmake --build build - name: Run tests if: always() From 625b752de0498c8f9ebe57cec51311c24ee4921a Mon Sep 17 00:00:00 2001 From: Samman Das <105208798+RayBreeze@users.noreply.github.com> Date: Tue, 7 Oct 2025 09:00:29 +0530 Subject: [PATCH 5/5] Update CI workflow to use only Windows environment Removed Ubuntu from the CI workflow matrix and the related dependency installation step. As Linux build is not currently available right now. --- .github/workflows/ci.yml | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index ec9dc31..5104ea2 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -11,15 +11,11 @@ jobs: runs-on: ${{ matrix.os }} strategy: matrix: - os: [ubuntu-latest, windows-latest] + os: [windows-latest] steps: - uses: actions/checkout@v4 - - name: Install dependencies - if: matrix.os == 'ubuntu-latest' - run: sudo apt-get install -y build-essential cmake - - name: Configure run: cmake -S . -B build -DCMAKE_BUILD_TYPE=Release