Skip to content

Conversation

@zhouang777
Copy link
Collaborator

  1. add linux build action.
  2. rename artifact according to git tag or git commit.

@zhouang777
Copy link
Collaborator Author

zhouang777 commented Aug 29, 2025

resolve this issue: #122

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Linux build support to the Bazel CLI build workflow and implements dynamic artifact naming based on git tags or commit hashes. The changes expand the CI pipeline from macOS-only to support both Linux and macOS builds.

Key changes:

  • Added a new Linux build job using Ubuntu 22.04
  • Implemented dynamic artifact naming using git tags for releases or commit hashes for regular pushes
  • Updated the macOS job to use the new naming scheme and upgraded the runner version

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

zhouang777 and others added 3 commits September 2, 2025 11:22
merge artifact name step

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- name: Build All
run: bazel build //...
- name: Copy and rename artifact
uses: ./.github/actions/copy-artifact
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI failed at this line

Copy link
Collaborator

@ValKmjolnir ValKmjolnir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhouang777 zhouang777 merged commit bccce36 into main Sep 21, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants