Skip to content

Resolve Issue #95 adding CI/CD#165

Open
bdtran2002 wants to merge 2 commits intoChicoState:mainfrom
bdtran2002:issue_#95
Open

Resolve Issue #95 adding CI/CD#165
bdtran2002 wants to merge 2 commits intoChicoState:mainfrom
bdtran2002:issue_#95

Conversation

@bdtran2002
Copy link

Issue #95
Added CI/CD for Task #153

Copilot AI review requested due to automatic review settings February 18, 2026 19:47
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

Adds a GitHub Actions workflow intended to compile the C++ program in CI for PRs targeting main and pushes to the devops branch, aligning with Issue #95.

Changes:

  • Added a compile GitHub Actions workflow triggered on pull_request to main and push to devops.
  • Workflow runs g++ -std=c++17 main.cpp on ubuntu-latest.

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

Comment on lines +10 to +21
install:
runs-on: ubuntu-latest
steps:
- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y -f build-essential g++ cmake
build:
needs: install
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

The install and build jobs run on separate fresh runners, so packages installed in install are not available in build. As written, needs: install only enforces ordering and adds time without affecting the build environment. Consider merging install+build into a single job (checkout -> apt-get -> g++) or install in the build job (or drop the install step if relying on ubuntu-latest toolchain).

Suggested change
install:
runs-on: ubuntu-latest
steps:
- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y -f build-essential g++ cmake
build:
needs: install
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y -f build-essential g++ cmake

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +7
name: compile

on:
push:
branches: [ devops ]
pull_request:
branches: [ main ]
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

PR description/Issue #95 mention adding a status badge for the compile workflow to README.md, but this PR only adds the workflow file and does not update README.md (current README has no badge). Either include the README badge change or update the PR description/linked issue scope accordingly.

Copilot uses AI. Check for mistakes.
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