Skip to content

Build/add workflow#6

Merged
ChingEnLin merged 5 commits intodevfrom
build/add_workflow
Aug 29, 2025
Merged

Build/add workflow#6
ChingEnLin merged 5 commits intodevfrom
build/add_workflow

Conversation

@ChingEnLin
Copy link
Copy Markdown
Owner

This pull request introduces a comprehensive CI/CD pipeline for the QueryPal project using GitHub Actions. It adds a detailed workflow for running backend and frontend tests, performing build verification, and uploading coverage reports. Additionally, it documents the new pipeline and removes the previous Docker build and push workflow.

CI/CD Pipeline Implementation and Documentation:

  • Added a new .github/workflows/ci.yml workflow that runs backend (Python) and frontend (Node.js) tests in parallel, checks code quality, uploads coverage reports to Codecov, and verifies frontend build artifacts. This workflow is triggered on pushes and pull requests to key branches.
  • Added a new .github/README.md file documenting the structure, jobs, and usage of the new CI/CD pipeline, including local testing instructions and coverage details.

Cleanup of Legacy Workflows:

  • Removed the old .github/workflows/docker-build-push.yml workflow, which previously handled Docker image builds and pushes to Azure Container Registry.

Copilot AI review requested due to automatic review settings August 29, 2025 07:54
@ChingEnLin ChingEnLin merged commit d7eebe0 into dev Aug 29, 2025
2 checks passed
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 introduces a comprehensive CI/CD pipeline using GitHub Actions to replace Docker-based workflows with test-focused automation. The changes implement parallel testing for both backend (Python) and frontend (Node.js) components with coverage reporting and build verification.

  • Adds a complete CI workflow with backend/frontend testing, linting, and coverage reporting
  • Introduces comprehensive documentation for the new CI/CD pipeline structure and local development
  • Removes legacy Docker build workflow that pushed to Azure Container Registry

Reviewed Changes

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

File Description
.github/workflows/docker-build-push.yml Removes legacy Docker build and push workflow for ACR
.github/workflows/ci.yml Adds comprehensive CI pipeline with parallel backend/frontend testing and build verification
.github/README.md Documents the new CI/CD pipeline structure, jobs, and local testing instructions
Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:1

  • [nitpick] The coverage percentages in the documentation may become outdated as the codebase evolves. Consider removing specific percentages or adding a note that these are approximate/historical values.
name: CI - Test Frontend and Backend

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

@ChingEnLin ChingEnLin deleted the build/add_workflow branch August 29, 2025 07:54
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