Skip to content

feat(ci): add selective backend/web/mobile checks with PR comments#450

Merged
Harxhit merged 10 commits into
Dev-Card:mainfrom
Harxhit:lint-proof
Jun 1, 2026
Merged

feat(ci): add selective backend/web/mobile checks with PR comments#450
Harxhit merged 10 commits into
Dev-Card:mainfrom
Harxhit:lint-proof

Conversation

@Harxhit
Copy link
Copy Markdown
Collaborator

@Harxhit Harxhit commented Jun 1, 2026

Summary

Adds a selective monorepo CI workflow that detects changed files and runs domain-specific checks for backend, web, and mobile applications. This improves CI efficiency by avoiding unnecessary checks and introduces automated PR result comments summarizing CI outcomes. Mobile emulator/device build validation is intentionally deferred for future work.


Type of Change

  • Bug fix
  • New feature
  • Refactor (no functional change)
  • UI / Design change
  • Tests only
  • Documentation
  • Infrastructure / DevOps
  • Security

What Changed

  • Added change-detection logic using GitHub API (pulls.listFiles) to identify modified backend, web, and mobile files.
  • Added selective CI jobs for backend, web, and mobile that run only when corresponding files change.
  • Added PR comment automation to summarize CI results across backend, web, and mobile jobs.

How to Test

  1. Open or update a PR with changes in apps/backend, apps/web, or apps/mobile.
  2. Verify only the relevant CI jobs run based on modified files.
  3. Confirm the workflow posts a PR comment summarizing CI job results.

Checklist

  • My code follows the project's coding style (pnpm -r run lint passes).
  • TypeScript compiles without errors (pnpm -r run typecheck).
  • I have added or updated tests for the changes I made.
  • All tests pass locally (pnpm -r run test).
  • I have updated documentation where necessary.
  • No new console.log or debug statements left in the code.
  • Breaking changes are documented in this PR description.

Screenshots / Recordings

N/A


Additional Context

  • CI currently validates backend, web, and mobile lint/build/test workflows selectively.
  • Mobile emulator/device execution and E2E validation are intentionally not included in this PR and are planned for future improvements.
  • This setup is designed to scale with the monorepo and reduce unnecessary CI execution time.

@Harxhit
Copy link
Copy Markdown
Collaborator Author

Harxhit commented Jun 1, 2026

@ShantKhatri Please check this.

@Harxhit Harxhit requested a review from ShantKhatri June 1, 2026 18:03
@Harxhit Harxhit added the gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. label Jun 1, 2026
@Harxhit Harxhit merged commit 55d7d56 into Dev-Card:main Jun 1, 2026
1 of 2 checks passed
@Harxhit Harxhit added level:critical High-impact or difficult contribution affecting core functionality. (+80 pts) quality:exceptional Outstanding contribution with excellent implementation quality. (×1.5 multiplier) type:testing Adds/improves tests (+10 pts) type:accessibility Accessibility improvements (+15 pts) type:performance Performance optimization (+15 pts) type:devops CI/CD, infra, deployment, workflow work (+15 pts) labels Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Required label for every approved PR. Gives the base +50 points and enables contribution tracking. level:critical High-impact or difficult contribution affecting core functionality. (+80 pts) quality:exceptional Outstanding contribution with excellent implementation quality. (×1.5 multiplier) type:accessibility Accessibility improvements (+15 pts) type:devops CI/CD, infra, deployment, workflow work (+15 pts) type:performance Performance optimization (+15 pts) type:testing Adds/improves tests (+10 pts)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants