Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 9, 2025

Replaces CircleCI with GitHub Actions for CI pipeline.

Changes

  • Added .github/workflows/ci.yml with test_debug and test_release jobs
    • Uses same Docker image (tschijnmo/drudge:base) as CircleCI
    • Triggers on push/PR to main/master branches
    • Includes explicit contents: read permissions
  • Updated README.md badge to point to GitHub Actions workflow
  • Removed .circleci/config.yml

Build and test process remains identical: CMake debug/release builds followed by test execution.

Original prompt

Migrate from CircleCI to GitHub Actions for CI.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits November 9, 2025 17:59
Co-authored-by: chenpeizhi <8114085+chenpeizhi@users.noreply.github.com>
Co-authored-by: chenpeizhi <8114085+chenpeizhi@users.noreply.github.com>
Co-authored-by: chenpeizhi <8114085+chenpeizhi@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate CI configuration from CircleCI to GitHub Actions Migrate from CircleCI to GitHub Actions Nov 9, 2025
Copilot AI requested a review from chenpeizhi November 9, 2025 18:04
@chenpeizhi chenpeizhi marked this pull request as ready for review November 9, 2025 21:21
Copilot AI review requested due to automatic review settings November 9, 2025 21:21
Copy link
Contributor

@chenpeizhi chenpeizhi left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link

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 migrates the CI/CD pipeline from CircleCI to GitHub Actions. The repository URL references are updated from tschijnmo to DrudgeCAS organization, and the legacy YouCompleteMe configuration file is removed.

  • Replaced CircleCI configuration with GitHub Actions workflow
  • Updated README badge to reflect the new CI system
  • Removed obsolete .ycm_extra_conf.py file

Reviewed Changes

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

File Description
README.md Updated CI badge from CircleCI to GitHub Actions with new repository URL
.ycm_extra_conf.py Removed legacy YouCompleteMe configuration file
.github/workflows/ci.yml Added new GitHub Actions workflow with debug and release test jobs
.circleci/config.yml Removed CircleCI configuration

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

run: |
mkdir cmake-test-debug
cd cmake-test-debug
cmake -DCMAKE_BUILD_TYPE=debug ..
Copy link

Copilot AI Nov 9, 2025

Choose a reason for hiding this comment

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

CMAKE_BUILD_TYPE values should be capitalized. Use 'Debug' instead of 'debug' to follow CMake conventions.

Copilot uses AI. Check for mistakes.
run: |
mkdir cmake-test-release
cd cmake-test-release
cmake -DCMAKE_BUILD_TYPE=release ..
Copy link

Copilot AI Nov 9, 2025

Choose a reason for hiding this comment

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

CMAKE_BUILD_TYPE values should be capitalized. Use 'Release' instead of 'release' to follow CMake conventions.

Suggested change
cmake -DCMAKE_BUILD_TYPE=release ..
cmake -DCMAKE_BUILD_TYPE=Release ..

Copilot uses AI. Check for mistakes.
@chenpeizhi chenpeizhi merged commit d9c6813 into master Nov 9, 2025
8 checks passed
@chenpeizhi chenpeizhi deleted the copilot/migrate-to-github-actions branch November 9, 2025 21:22
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.

2 participants