Skip to content

GitHub actions and general CI maintenance #251

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.

Sign up for GitHub

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

Jump to bottom
Draft
ethanglaser wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

GitHub actions and general CI maintenance #251

ethanglaser wants to merge 7 commits into from

Conversation

@ethanglaser
Copy link
Member

@ethanglaser ethanglaser commented Dec 18, 2025 •
edited
Loading

Several CI related cleanup/refactoring tasks, functionality should largely be unaffected. So far:

  • Replaces renovate with dependabot to align with private source repo, with monthly scans for actions updates
  • Uses version tags for all actions for improved security and updates
  • Migrate CI-related bash scripts from docker to .github/scripts
  • Remove unnecessary chmod on above scripts
  • Update cpp bindings and shared examples CMakeLists.txt to reference latest release binaries
  • Throw error if ctests aren't found in shared/bindings validation
  • Run Faiss-SVS examples for validation in addition to tests

@ethanglaser ethanglaser changed the title Dev/eglaser ghactions refactoring GitHub actions and general CI maintenance Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mihaic mihaic Awaiting requested review from mihaic mihaic will be requested when the pull request is marked ready for review mihaic is a code owner

@homksei homksei Awaiting requested review from homksei homksei will be requested when the pull request is marked ready for review homksei is a code owner

@yuejiaointel yuejiaointel Awaiting requested review from yuejiaointel yuejiaointel will be requested when the pull request is marked ready for review yuejiaointel is a code owner

@ahuber21 ahuber21 Awaiting requested review from ahuber21 ahuber21 will be requested when the pull request is marked ready for review ahuber21 is a code owner

@ibhati ibhati Awaiting requested review from ibhati ibhati will be requested when the pull request is marked ready for review ibhati is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ethanglaser