Skip to content

update README for new systems, add GitHub Actions CI and remove GitLab CI support#7

Merged
HehuiZheng merged 4 commits intomainfrom
update-docs
Mar 18, 2026
Merged

update README for new systems, add GitHub Actions CI and remove GitLab CI support#7
HehuiZheng merged 4 commits intomainfrom
update-docs

Conversation

@Yasu31
Copy link
Copy Markdown
Contributor

@Yasu31 Yasu31 commented Mar 11, 2026

No description provided.

Copy link
Copy Markdown
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 updates project onboarding and CI to reflect a move from GitLab CI to GitHub Actions, and modernizes the README’s platform/setup guidance.

Changes:

  • Add a GitHub Actions workflow to build the project on Ubuntu and fetch submodules.
  • Remove legacy GitLab CI configuration and the Dockerfile previously used for CI hints.
  • Update README platform/support notes and dependency install instructions to reference the new CI workflow.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
README.md Updates supported platforms and setup guidance; points users to the new GitHub Actions workflow for dependency hints.
Dockerfile Removed (previously provided Ubuntu build dependency hints).
.gitlab-ci.yml Removed GitLab CI pipeline configuration.
.github/workflows/ci.yml Adds GitHub Actions CI build on Ubuntu, including dependency installation and compilation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


jobs:
build:
runs-on: ubuntu-latest
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

Using ubuntu-latest can change the underlying OS version over time and make builds non-reproducible. Consider pinning to a specific runner image (e.g., ubuntu-22.04 or ubuntu-24.04) and updating intentionally when needed.

Suggested change
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

Copilot uses AI. Check for mistakes.

steps:
- name: Checkout repository
uses: actions/checkout@v4
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

For stronger supply-chain security, consider pinning GitHub Actions to a commit SHA instead of a movable tag like actions/checkout@v4 (tags can be retargeted).

Suggested change
uses: actions/checkout@v4
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@HehuiZheng HehuiZheng merged commit d0adddd into main Mar 18, 2026
2 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