Skip to content

chore: add Prettier formatting validation to frontend PRs#542

Open
DevyanshiBansal wants to merge 2 commits into
param20h:devfrom
DevyanshiBansal:chore/add-prettier-validation
Open

chore: add Prettier formatting validation to frontend PRs#542
DevyanshiBansal wants to merge 2 commits into
param20h:devfrom
DevyanshiBansal:chore/add-prettier-validation

Conversation

@DevyanshiBansal

@DevyanshiBansal DevyanshiBansal commented Jun 9, 2026

Copy link
Copy Markdown

📋 PR Checklist

Thank you for contributing to PDF-Assistant-RAG! 🎉
Please fill out this template before submitting. PRs without it filled in will be closed.


🔗 Related Issue

Closes #455


📝 What does this PR do?

Added a quick workflow to enforce Prettier formatting on the frontend. It automatically checks TS, CSS, and Markdown files on incoming PRs to dev so we don't have to worry about style inconsistencies during code reviews.


🗂️ Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactor / code cleanup
  • 📝 Documentation update
  • 🎨 UI / styling change
  • [x ] ⚙️ CI / tooling / config change
  • 🧪 Tests

🧪 How was this tested?

  • Ran the backend locally (uvicorn app.main:app --reload)
  • Ran the frontend locally (npm run dev inside frontend/)
  • Tested the affected API endpoints manually
  • Added / updated tests
  • Note: I verified the YAML syntax and tested the npx prettier --check command locally to ensure it correctly flags unformatted files without altering them.

📸 Screenshots (if UI change)

N/A


⚠️ Anything to flag for reviewers?


✅ Self-Review Checklist

  • My branch is based on dev, not main
  • I have not added any secrets / API keys
  • I have not modified main branch or any HuggingFace deployment config
  • My code follows the existing style (no unnecessary formatting changes)
  • I have updated relevant docs / comments if needed

@DevyanshiBansal DevyanshiBansal requested a review from param20h as a code owner June 9, 2026 16:36
Copilot AI review requested due to automatic review settings June 9, 2026 16:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a GitHub Actions workflow to validate Prettier formatting for the frontend on pull requests targeting dev.

Changes:

  • Introduces a new workflow triggered on pull_request to dev
  • Sets up Node.js 20 and runs a Prettier --check over common frontend file types

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

Comment thread .github/workflows/prettier-check.yml
Comment thread .github/workflows/prettier-check.yml
Comment thread .github/workflows/prettier-check.yml Outdated
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.

chore(ci): Add Prettier formatting validation to frontend PRs

3 participants