Skip to content

ci: improve pipeline feedback and deploy health - #27

Merged
TahubCS merged 1 commit into
mainfrom
feature/pipeline-feedback
Nov 9, 2026
Merged

TahubCS merged 1 commit into
mainfrom
feature/pipeline-feedback

Conversation

@vinay-moorthy

Copy link
Copy Markdown
Collaborator

Summary

Improve CI feedback and API deployment availability without changing application behavior or database schema.

Changes

  • Enable NuGet caching in the verify and deploy-api jobs, keyed by app/api/packages.lock.json; locked restore remains enabled.
  • Upload available Playwright reports and test results as a seven-day artifact when verify fails.
  • After the Render deploy hook, poll the public API /health endpoint for up to five minutes. Each request is capped at two seconds with eight-second pauses between attempts, keeping the worst case under five minutes.
  • Document the design, CI behavior, and handoff.

Validation

Passed locally:

  • npm run lint
  • npm run typecheck
  • dotnet format app/api --verify-no-changes --no-restore
  • npm run build
  • npm run build:api
  • npm test (409 passed)
  • Workflow YAML and structural checks; git diff --check

GitHub Actions has not run on this branch yet and will run on this PR.

Deployment and Limitations

  • No database migrations, application secrets, or production settings were changed.
  • deploy-api remains restricted to pushes to main and runs after verify and api-image pass.
  • /health is a liveness check only. It does not prove the exact requested commit is serving or that the database is ready. This health check runs after a merge to main; no live Render deployment was triggered during local validation.

@vercel

vercel Bot commented Nov 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
web Ready Ready Preview Nov 8, 2026 9:38pm UTC

@coderabbitai

coderabbitai Bot commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2cf90982-6a9d-4f00-a379-386c4a0a4381
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@TahubCS
TahubCS merged commit fd89908 into main Nov 9, 2026
6 checks passed

This branch was successfully deployed

1 active deployment
Preview — 478752c1 Deployed Nov 8, 2026 by vercel[bot]
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