Skip to content

chore: apply Prettier formatting and add formatting config #1347

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Yasir-Rafique
Copy link
Contributor

  • Adds Prettier as a dev dependency and includes a .prettierrc configuration file.
  • Applies Prettier formatting to the entire codebase (JavaScript files).
  • This commit contains only formatting changes and Prettier setup.
  • No code logic or functionality was changed.

Related to #1341.

@CLAassistant
Copy link

CLAassistant commented Jul 12, 2025

CLA assistant check
All committers have signed the CLA.

@Yasir-Rafique
Copy link
Contributor Author

Hi! 👋

This PR implements the short-term solution discussed in #1341 by adding Prettier as a dev dependency, creating a .prettierrc config, and formatting the entire codebase for consistency.

If this approach works for you and the PR is merged, I’d be happy to move forward with the mid-term solution (documenting formatting in CONTRIBUTING.md, adding format scripts, etc.) and the long-term solution (integrating linting into CI) as next steps.

Let me know your thoughts or any preferences for the Prettier configuration!

@KernelDeimos
Copy link
Contributor

Hi, this is a guaranteed merge conflict with everything, so we need to carefully time when we merge formatting changes or even incrementally merge different portions. For this reason, this PR should only contain the config, not the formatting changes.

@Yasir-Rafique Yasir-Rafique force-pushed the chore/add-prettier-and-formatting branch from 898ec55 to 432dae1 Compare July 16, 2025 04:52
@Yasir-Rafique
Copy link
Contributor Author

As per the feedback, this PR now only adds the Prettier configuration files (.prettierrc and .prettierignore) and does not apply any formatting changes yet.

@KernelDeimos KernelDeimos merged commit e918490 into HeyPuter:main Jul 24, 2025
1 check passed
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.

3 participants