Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

Commit 15a256b

Browse files
authored
Merge pull request #326 from janhq/fix/ci_disable_debug
fix: Remove -DDEBUG=ON
2 parents a4a1d1a + 1048e5a commit 15a256b

File tree

1 file changed

+83
-23
lines changed

1 file changed

+83
-23
lines changed

.github/workflows/build.yml

Lines changed: 83 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,49 @@ name: CI
22

33
on:
44
schedule:
5-
- cron: '0 20 * * *' # At 8 PM UTC, which is 3 AM UTC+7
5+
- cron: "0 20 * * *" # At 8 PM UTC, which is 3 AM UTC+7
66
push:
77
branches:
88
- main
99
tags: ["v[0-9]+.[0-9]+.[0-9]+"]
10-
paths: ['.github/scripts/**','.github/workflows/build.yml', '**/CMakeLists.txt', '**/Makefile', '**/*.h', '**/*.hpp', '**/*.c', '**/*.cpp', '**/*.cu', '**/*.cc', '**/*.cxx', "llama.cpp", '!docs/**', '!.gitignore', "!README.md"]
10+
paths:
11+
[
12+
".github/scripts/**",
13+
".github/workflows/build.yml",
14+
"**/CMakeLists.txt",
15+
"**/Makefile",
16+
"**/*.h",
17+
"**/*.hpp",
18+
"**/*.c",
19+
"**/*.cpp",
20+
"**/*.cu",
21+
"**/*.cc",
22+
"**/*.cxx",
23+
"llama.cpp",
24+
"!docs/**",
25+
"!.gitignore",
26+
"!README.md",
27+
]
1128
pull_request:
1229
types: [opened, synchronize, reopened]
13-
paths: ['.github/scripts/**','.github/workflows/build.yml', '**/CMakeLists.txt', '**/Makefile', '**/*.h', '**/*.hpp', '**/*.c', '**/*.cpp', '**/*.cu', '**/*.cc', '**/*.cxx', "llama.cpp", '!docs/**', '!.gitignore', "!README.md"]
30+
paths:
31+
[
32+
".github/scripts/**",
33+
".github/workflows/build.yml",
34+
"**/CMakeLists.txt",
35+
"**/Makefile",
36+
"**/*.h",
37+
"**/*.hpp",
38+
"**/*.c",
39+
"**/*.cpp",
40+
"**/*.cu",
41+
"**/*.cc",
42+
"**/*.cxx",
43+
"llama.cpp",
44+
"!docs/**",
45+
"!.gitignore",
46+
"!README.md",
47+
]
1448
workflow_dispatch:
1549

1650
env:
@@ -105,7 +139,7 @@ jobs:
105139
ldd --version
106140
./install_deps.sh
107141
mkdir build && cd build
108-
cmake -DDEBUG=ON -DLLAMA_NATIVE=OFF -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} ..
142+
cmake -DLLAMA_NATIVE=OFF -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} ..
109143
make -j $(nproc)
110144
ls -la
111145
@@ -148,7 +182,7 @@ jobs:
148182
contents: write
149183
strategy:
150184
matrix:
151-
cuda: ['12-0', '11-4']
185+
cuda: ["12-0", "11-4"]
152186

153187
steps:
154188
- name: Clone
@@ -162,9 +196,9 @@ jobs:
162196
run: |
163197
./install_deps.sh
164198
mkdir build && cd build
165-
cmake -DDEBUG=ON -DLLAMA_CUBLAS=ON -DLLAMA_NATIVE=OFF -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} ..
199+
cmake -DLLAMA_CUBLAS=ON -DLLAMA_NATIVE=OFF -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} ..
166200
make -j $(nproc)
167-
ls -la
201+
ls -la
168202
169203
- name: Package
170204
shell: bash
@@ -224,8 +258,8 @@ jobs:
224258
mkdir build && cd build
225259
cmake -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} ..
226260
CC=gcc-8 make -j $(sysctl -n hw.ncp)
227-
ls -la
228-
261+
ls -la
262+
229263
- name: Package
230264
shell: bash
231265
run: |
@@ -284,8 +318,8 @@ jobs:
284318
mkdir build && cd build
285319
cmake -DNITRO_VERSION=${{ needs.set-nitro-version.outputs.version }} -DLLAMA_METAL=OFF ..
286320
CC=gcc-8 make -j $(sysctl -n hw.ncp)
287-
ls -la
288-
321+
ls -la
322+
289323
- name: Package
290324
shell: bash
291325
run: |
@@ -326,7 +360,7 @@ jobs:
326360

327361
steps:
328362
- name: Clone
329-
363+
330364
id: checkout
331365
uses: actions/checkout@v3
332366
with:
@@ -339,7 +373,7 @@ jobs:
339373
silent: true
340374
env:
341375
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
342-
376+
343377
- name: actions-setup-cmake
344378
uses: jwlawson/actions-setup-cmake@v1.14.1
345379

@@ -397,7 +431,7 @@ jobs:
397431

398432
strategy:
399433
matrix:
400-
cuda: ['12-0', '11-4']
434+
cuda: ["12-0", "11-4"]
401435

402436
steps:
403437
- name: Setup VSWhere.exe
@@ -407,7 +441,7 @@ jobs:
407441
silent: true
408442
env:
409443
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
410-
444+
411445
- name: actions-setup-cmake
412446
uses: jwlawson/actions-setup-cmake@v1.14.1
413447

@@ -424,10 +458,10 @@ jobs:
424458
silent: true
425459
env:
426460
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
427-
461+
428462
- uses: actions/setup-dotnet@v3
429463
with:
430-
dotnet-version: '6.0.x'
464+
dotnet-version: "6.0.x"
431465

432466
- name: Build
433467
id: cmake_build
@@ -464,7 +498,7 @@ jobs:
464498
# run: |
465499
# cd .\build\Release
466500
# ..\..\.github\scripts\e2e-test-windows.bat .\nitro.exe ${{ env.MODEL_URL }}
467-
501+
468502
- uses: actions/upload-release-asset@v1.0.1
469503
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')
470504
env:
@@ -474,10 +508,18 @@ jobs:
474508
asset_path: ./nitro.tar.gz
475509
asset_name: nitro-${{ needs.create-draft-release.outputs.version }}-win-amd64-cuda-${{ matrix.cuda }}.tar.gz
476510
asset_content_type: application/gzip
477-
511+
478512
update_release_draft:
479513
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/')
480-
needs: [ubuntu-amd64-build, ubuntu-amd64-cuda-build, macOS-M-build, macOS-Intel-build, windows-amd64-build, windows-amd64-cuda-build]
514+
needs:
515+
[
516+
ubuntu-amd64-build,
517+
ubuntu-amd64-cuda-build,
518+
macOS-M-build,
519+
macOS-Intel-build,
520+
windows-amd64-build,
521+
windows-amd64-cuda-build,
522+
]
481523
permissions:
482524
contents: write
483525
pull-requests: write
@@ -489,7 +531,16 @@ jobs:
489531

490532
noti-discord-nightly:
491533
if: always() && github.event_name == 'schedule' && (needs.create-draft-release.result == 'success' || needs.create-draft-release.result == 'skipped') && needs.ubuntu-amd64-build.result == 'success' && needs.ubuntu-amd64-cuda-build.result == 'success' && needs.macOS-M-build.result == 'success' && needs.macOS-Intel-build.result == 'success' && needs.windows-amd64-build.result == 'success' && needs.windows-amd64-cuda-build.result == 'success'
492-
needs: [create-draft-release, ubuntu-amd64-build, ubuntu-amd64-cuda-build, macOS-M-build, macOS-Intel-build, windows-amd64-build, windows-amd64-cuda-build]
534+
needs:
535+
[
536+
create-draft-release,
537+
ubuntu-amd64-build,
538+
ubuntu-amd64-cuda-build,
539+
macOS-M-build,
540+
macOS-Intel-build,
541+
windows-amd64-build,
542+
windows-amd64-cuda-build,
543+
]
493544
runs-on: ubuntu-latest
494545
steps:
495546
- name: Checkout code
@@ -516,7 +567,16 @@ jobs:
516567

517568
noti-discord-manual:
518569
if: always() && github.event_name == 'workflow_dispatch' && (needs.create-draft-release.result == 'success' || needs.create-draft-release.result == 'skipped') && needs.ubuntu-amd64-build.result == 'success' && needs.ubuntu-amd64-cuda-build.result == 'success' && needs.macOS-M-build.result == 'success' && needs.macOS-Intel-build.result == 'success' && needs.windows-amd64-build.result == 'success' && needs.windows-amd64-cuda-build.result == 'success'
519-
needs: [create-draft-release, ubuntu-amd64-build, ubuntu-amd64-cuda-build, macOS-M-build, macOS-Intel-build, windows-amd64-build, windows-amd64-cuda-build]
570+
needs:
571+
[
572+
create-draft-release,
573+
ubuntu-amd64-build,
574+
ubuntu-amd64-cuda-build,
575+
macOS-M-build,
576+
macOS-Intel-build,
577+
windows-amd64-build,
578+
windows-amd64-cuda-build,
579+
]
520580
runs-on: ubuntu-latest
521581
steps:
522582
- name: Checkout code
@@ -541,4 +601,4 @@ jobs:
541601
git commit -m "${GITHUB_REPOSITORY}: Update README.md with nightly build artifact URL"
542602
git -c http.extraheader="AUTHORIZATION: bearer ${{ secrets.PAT_SERVICE_ACCOUNT }}" push origin HEAD:main
543603
env:
544-
GITHUB_RUN_ID: ${{ github.run_id }}
604+
GITHUB_RUN_ID: ${{ github.run_id }}

0 commit comments

Comments
 (0)