Skip to content

Update: Added K8s & Jenkins CI/CD#8

Merged
hoangsonww merged 1 commit into
masterfrom
feat/add-ci-cd-tests
Jul 4, 2025
Merged

Update: Added K8s & Jenkins CI/CD#8
hoangsonww merged 1 commit into
masterfrom
feat/add-ci-cd-tests

Conversation

@hoangsonww

Copy link
Copy Markdown
Owner

Pull Request Template

Overview

This pull request implements the CI/CD pipeline for the LMS application. It includes:

  • Formatting and linting setup
  • Parallel backend (pytest) and frontend (Jest) testing
  • Frontend code coverage collection
  • Docker build & push to GHCR
  • Simple deployment step messaging
  • Final completion notification

Files Changed

  • .github/workflows/ci.yml (added full pipeline config)
  • Added test specs under frontend/__tests__/
  • Minor updates to package.json scripts
  • Added .dockerignore and adjusted Dockerfile for backend & frontend

Testing

  • Ran all jobs locally in dry-run mode
  • Verified pytest -q works for the backend
  • Verified npm test and npm run coverage for the frontend
  • Confirmed Docker images built and pushed successfully on GHCR
  • Confirmed GitHub Actions workflow passes on PR event

Screenshots (if applicable)

N/A (pipeline config)

Notes to Reviewers

  • Please check the Docker build steps closely — I set explicit context and file paths
  • Deployment is currently a simple echo message; future production deployment commands can replace these
  • If you want, you can test the pipeline by pushing to a feature branch to verify parallel jobs

Deployment Notes

  • Ensure secrets.GITHUB_TOKEN is available for GHCR push
  • Ensure Render and Vercel accounts are ready for actual deployment once the placeholders are swapped with real commands

@hoangsonww hoangsonww requested a review from Copilot July 4, 2025 14:04
@hoangsonww hoangsonww self-assigned this Jul 4, 2025
@hoangsonww hoangsonww added bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request labels Jul 4, 2025
@netlify

netlify Bot commented Jul 4, 2025

Copy link
Copy Markdown

Deploy Preview for fullstack-learning-management-system canceled.

Name Link
🔨 Latest commit 34d7949
🔍 Latest deploy log https://app.netlify.com/projects/fullstack-learning-management-system/deploys/6867df8635a73a00087a625e

@hoangsonww hoangsonww merged commit 53d6b06 into master Jul 4, 2025
6 of 7 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

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 documentation formatting in the README to use proper blockquote lines for the “IMPORTANT” note.

  • Split the inline admonition into two separate blockquote lines
  • Ensured consistency of note formatting
Comments suppressed due to low confidence (1)

README.md:150

  • [nitpick] GitHub-flavored Markdown doesn’t recognize the [!IMPORTANT] admonition syntax. Consider merging these two lines into a single blockquote with bold text, for example: > **IMPORTANT**: Our backend server may spin down due to inactivity, so you may experience delays in loading data initially…
> [!IMPORTANT] 

@vercel

vercel Bot commented Jul 4, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
learning-management-system-fullstack ⬜️ Ignored (Inspect) Jul 4, 2025 2:05pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

2 participants