Skip to content

Add CI/CD workflow for multi-OS binaries#23

Merged
jamylak merged 20 commits intomainfrom
copilot/add-cicd-workflow-for-builds
Jan 13, 2026
Merged

Add CI/CD workflow for multi-OS binaries#23
jamylak merged 20 commits intomainfrom
copilot/add-cicd-workflow-for-builds

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 17, 2025

Summary

Binaries on release

Copilot AI and others added 3 commits December 17, 2025 07:21
Co-authored-by: jamylak <1940205+jamylak@users.noreply.github.com>
Co-authored-by: jamylak <1940205+jamylak@users.noreply.github.com>
Co-authored-by: jamylak <1940205+jamylak@users.noreply.github.com>
@jamylak
Copy link
Copy Markdown
Owner

jamylak commented Jan 13, 2026

@copilot finish this PR and make the workflows only run on release or manual dispatch, not on pushes or PRs

…d security permissions

Co-authored-by: jamylak <1940205+jamylak@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 13, 2026

@copilot finish this PR and make the workflows only run on release or manual dispatch, not on pushes or PRs

Done! The workflow now only runs on release tags (v*) or manual dispatch. I also added explicit security permissions to all jobs. Commit: 1eed3e3

Copilot AI changed the title [WIP] Add CI/CD workflow for multi-OS binaries Add CI/CD workflow for multi-OS binaries Jan 13, 2026
@jamylak
Copy link
Copy Markdown
Owner

jamylak commented Jan 13, 2026

@copilot update this so that edits to this release binary workflow should not trigger all the other workflows to run.

…other workflows

Co-authored-by: jamylak <1940205+jamylak@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 13, 2026

@copilot update this so that edits to this release binary workflow should not trigger all the other workflows to run.

Done! Added paths-ignore filters to test.yml, smoketest.yml, and windows-test.yml to exclude .github/workflows/build.yml. Now edits to the build workflow won't trigger the other CI workflows. Commit: eb241be

11,
       inlined from ‘constexpr void
fmt::v9::detail::bigint::assign_pow10(int)’ at
/usr/include/fmt/format.h:2893:11,
       inlined from ‘constexpr void
fmt::v9::detail::bigint::assign_pow10(int)’ at
/usr/include/fmt/format.h:2884:24,
       inlined from ‘constexpr void
fmt::v9::detail::format_dragon(basic_fp<__int128 unsigned>, unsigned
int, int, buffer<char>&,
    int&)’ at /usr/include/fmt/format.h:3011:29:
   /usr/include/c++/13/bits/stl_algobase.h:437:30: error: ‘void*
__builtin_memmove(void*, const void*, long unsigned int)’
   forming offset 4 is out of the bounds [0, 4] [-Werror=array-bounds=]
     437 |             __builtin_memmove(__result, __first, sizeof(_Tp)
* _Num);
         |
~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1plus: all warnings being treated as errors
   gmake[2]: *** [CMakeFiles/vsdf.dir/build.make:121:
CMakeFiles/vsdf.dir/src/online_sdf_renderer.cpp.o] Error 1
   gmake[1]: *** [CMakeFiles/Makefi

    inlined from ‘constexpr void fmt::v9::detail::format_dragon(basic_fp<__int128 unsigned>, unsigned int, int, buffer<char>&, int&)’ at /usr/include/fmt/format.h:3011:29:
/usr/include/c++/13/bits/stl_algobase.h:437:30: error: ‘void* __builtin_memmove(void*, const void*, long unsigned int)’ forming offset 4 is out of the bounds [0, 4] [-Werror=array-bounds=]
  437 |             __builtin_memmove(__result, __first, sizeof(_Tp) * _Num);
      |             ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors
[9/10] Building CXX object CMakeFiles/vsdf.dir/src/ffmpeg_encoder.cpp.o
ninja: build stopped: subcommand failed.
Error: Process completed with exit code 1.

https://github.com/jamylak/vsdf/actions/runs/20944451445/job/60184584605?pr=23

Due to how we treat warnings as errors, somehow it finds this issue in
fmt/spdlog libraries
@jamylak jamylak marked this pull request as ready for review January 13, 2026 04:23
Copilot AI review requested due to automatic review settings January 13, 2026 04:23
@jamylak
Copy link
Copy Markdown
Owner

jamylak commented Jan 13, 2026

Will just merge for now in this state and address issues in a future PR.
After merging it wont retrigger all actions when updating this one release action

@jamylak jamylak review requested due to automatic review settings January 13, 2026 04:23
@jamylak jamylak merged commit 05bebce into main Jan 13, 2026
8 of 14 checks passed
@jamylak jamylak deleted the copilot/add-cicd-workflow-for-builds branch January 13, 2026 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants