[PR #1902] [CLOSED] chore: format all files with prettier #4699

Closed
opened 2026-04-20 09:02:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/1902
Author: @Fredkiss3
Created: 11/21/2025
Status: Closed

Base: devHead: chore/fix-formatting


📝 Commits (10+)

  • c85ed2f install prettier
  • 72b6735 🙈 ignore files for formatting
  • cdf27d5 🎨 format all files
  • 4da2f45 👷 add jobs for checking formatting
  • 8338cfe 🎨 fix formatting
  • 4bb6f4f 👷update CI format job
  • 8f6c39d 👷 recommend running npm run format
  • 2cf5dec ⚗️ test with formatting error
  • 283a64e 👷fix job
  • 2131cd3 👷refactor CI format check

📊 Changes

549 files changed (+8396 additions, -6255 deletions)

View changed files

📝 .eslintrc.json (+1 -4)
.github/workflows/check-format.yml (+28 -0)
.prettierignore (+12 -0)
.vscode/extensions.json (+3 -0)
.vscode/settings.json (+22 -0)
📝 components.json (+1 -1)
📝 drizzle.pg.config.ts (+1 -3)
📝 drizzle.sqlite.config.ts (+1 -3)
📝 esbuild.mjs (+62 -33)
📝 eslint.config.js (+16 -16)
📝 package-lock.json (+6 -34)
📝 package.json (+5 -3)
📝 postcss.config.mjs (+2 -2)
📝 server/auth/password.ts (+3 -3)
📝 server/auth/passwordSchema.ts (+6 -3)
📝 server/auth/sessions/newt.ts (+9 -13)
📝 server/auth/sessions/olm.ts (+9 -13)
📝 server/auth/sessions/verifySession.ts (+5 -2)
📝 server/cleanup.ts (+1 -1)
📝 server/db/countries.ts (+507 -507)

...and 80 more files

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

Main changes:

Other changes:

  • Added a job that checks that all files in are formatted correctly with prettier
  • Added a job to run formatting accross the whole repo: npm run format
  • Added .vscode/settings.json with some default options: setting prettier as the default formatter, and formatting on save
  • Added .vscode/extensions.json with recommendations for new contributors recommending them to install prettier extension if not installed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/fosrl/pangolin/pull/1902 **Author:** [@Fredkiss3](https://github.com/Fredkiss3) **Created:** 11/21/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `chore/fix-formatting` --- ### 📝 Commits (10+) - [`c85ed2f`](https://github.com/fosrl/pangolin/commit/c85ed2f39040e0821e0374cd252f48b64d5b355c) ➕install prettier - [`72b6735`](https://github.com/fosrl/pangolin/commit/72b6735944dfcbcf6c1bcb274879b9e904e2cbeb) 🙈 ignore files for formatting - [`cdf27d5`](https://github.com/fosrl/pangolin/commit/cdf27d5a4434fefec2e6774b6e6d28cbb1f214bb) 🎨 format all files - [`4da2f45`](https://github.com/fosrl/pangolin/commit/4da2f45627bddbb85f87cadd62a4fca5e1d20823) 👷 add jobs for checking formatting - [`8338cfe`](https://github.com/fosrl/pangolin/commit/8338cfe6efe4ad85c5639cc1068f98a67b011c8e) 🎨 fix formatting - [`4bb6f4f`](https://github.com/fosrl/pangolin/commit/4bb6f4fb573cb13ce7e1b7103ebd88cbf2d5d173) 👷update CI format job - [`8f6c39d`](https://github.com/fosrl/pangolin/commit/8f6c39d1be6ebba907083d688d1a0c57a30fd4a3) 👷 recommend running `npm run format` - [`2cf5dec`](https://github.com/fosrl/pangolin/commit/2cf5dec3cb86538aecaa3275514b6a7eee0574f2) ⚗️ test with formatting error - [`283a64e`](https://github.com/fosrl/pangolin/commit/283a64e6377de93e8fa9b476b2d0859eeb16c702) 👷fix job - [`2131cd3`](https://github.com/fosrl/pangolin/commit/2131cd3e6ab436e76816ece0c646597630bfe151) 👷refactor CI format check ### 📊 Changes **549 files changed** (+8396 additions, -6255 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.json` (+1 -4) ➕ `.github/workflows/check-format.yml` (+28 -0) ➕ `.prettierignore` (+12 -0) ➕ `.vscode/extensions.json` (+3 -0) ➕ `.vscode/settings.json` (+22 -0) 📝 `components.json` (+1 -1) 📝 `drizzle.pg.config.ts` (+1 -3) 📝 `drizzle.sqlite.config.ts` (+1 -3) 📝 `esbuild.mjs` (+62 -33) 📝 `eslint.config.js` (+16 -16) 📝 `package-lock.json` (+6 -34) 📝 `package.json` (+5 -3) 📝 `postcss.config.mjs` (+2 -2) 📝 `server/auth/password.ts` (+3 -3) 📝 `server/auth/passwordSchema.ts` (+6 -3) 📝 `server/auth/sessions/newt.ts` (+9 -13) 📝 `server/auth/sessions/olm.ts` (+9 -13) 📝 `server/auth/sessions/verifySession.ts` (+5 -2) 📝 `server/cleanup.ts` (+1 -1) 📝 `server/db/countries.ts` (+507 -507) _...and 80 more files_ </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description **Main changes:** - format all files with prettier. Included files are: - everything in src - everything in server - some of the files located at the root directory (excluding yaml and markdown files), refer `.prettierignore` file to see excluded files: https://github.com/Fredkiss3/pangolin/blob/chore/fix-formatting/.prettierignore **Other changes:** - Added a job that checks that all files in are formatted correctly with prettier - Added a job to run formatting accross the whole repo: `npm run format` - Added `.vscode/settings.json` with some default options: setting prettier as the default formatter, and formatting on save - Added `.vscode/extensions.json` with recommendations for new contributors recommending them to install prettier extension if not installed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-20 09:02:45 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#4699