Skip to content

chore(prettier): deduplicate frontend/backend config#343

Merged
leowla merged 3 commits intomasterfrom
chore/prettier-config
Apr 5, 2026
Merged

chore(prettier): deduplicate frontend/backend config#343
leowla merged 3 commits intomasterfrom
chore/prettier-config

Conversation

@leowla
Copy link
Copy Markdown
Member

@leowla leowla commented Apr 2, 2026

Describe the issue

Frontend and backend have the same prettier config. Potential for drifting rules.

Describe the solution

Use single root prettier config.

Risk

Maybe CI will fail

Definition of Done

  • Code peer-reviewed
  • Wiki Documentation is written and up to date
  • Unit tests written and passing
    - [ ] Integration tests written and passing
  • Continuous Integration build passing
    - [ ] Acceptance criteria met
    - [ ] Deployed to production environment

Reviewed By

Who reviewed your PR - for commit history once merged

@leowla leowla self-assigned this Apr 2, 2026
@leowla leowla requested a review from Copilot April 2, 2026 04:52
Copy link
Copy Markdown

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 removes duplicated Prettier configuration in the frontend/ and backend/ packages and centralizes formatting rules in a single repository-root Prettier config to reduce the chance of rule drift between the two codebases.

Changes:

  • Deleted frontend/.prettierrc.js
  • Deleted backend/.prettierrc.js
  • Added a root .prettierrc.json with the shared formatting rules

Reviewed changes

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

File Description
frontend/.prettierrc.js Removes the frontend-local Prettier config in favor of a shared root config.
backend/.prettierrc.js Removes the backend-local Prettier config in favor of a shared root config.
.prettierrc.json Introduces the shared Prettier configuration for the repo.

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

`prettify` project command does not exist. Correct command is `prettier:write`.
@leowla leowla added documentation Improvements or additions to documentation frontend backend labels Apr 2, 2026
Copy link
Copy Markdown
Contributor

@harbassan harbassan left a comment

Choose a reason for hiding this comment

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

sweet, good catch

Copy link
Copy Markdown
Contributor

@hazikchaudhry hazikchaudhry left a comment

Choose a reason for hiding this comment

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

nice, looks good

@leowla leowla merged commit 895aee7 into master Apr 5, 2026
5 checks passed
@leowla leowla deleted the chore/prettier-config branch April 5, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend documentation Improvements or additions to documentation frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants