[PR #2253] [MERGED] ⬆️ (prettier) upgrade, fix issues and enable for jsx files #34400

Closed
opened 2026-04-20 21:34:35 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2253
Author: @MatissJanis
Created: 1/19/2024
Status: Merged
Merged: 1/19/2024
Merged by: @MatissJanis

Base: masterHead: matiss/prettier


📝 Commits (2)

  • 021e606 ⬆️ (prettier) upgrade, fix issues and enable for jsx files
  • 6a4eb14 Use setupTests.js for tests

📊 Changes

49 files changed (+362 additions, -295 deletions)

View changed files

📝 .eslintrc.js (+2 -0)
📝 package.json (+4 -3)
📝 packages/crdt/src/crdt/timestamp.ts (+4 -4)
📝 packages/desktop-client/src/components/Notifications.tsx (+10 -10)
📝 packages/desktop-client/src/components/Titlebar.tsx (+10 -10)
📝 packages/desktop-client/src/components/accounts/Balance.jsx (+2 -2)
📝 packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/budget/BudgetCategories.jsx (+2 -2)
📝 packages/desktop-client/src/components/budget/MobileBudgetTable.jsx (+2 -2)
📝 packages/desktop-client/src/components/budget/report/budgetsummary/Saved.tsx (+2 -2)
📝 packages/desktop-client/src/components/budget/util.ts (+2 -2)
📝 packages/desktop-client/src/components/common/Stack.tsx (+15 -12)
📝 packages/desktop-client/src/components/filters/FiltersMenu.jsx (+6 -6)
📝 packages/desktop-client/src/components/modals/GoCardlessExternalMsg.tsx (+4 -4)
📝 packages/desktop-client/src/components/modals/ImportTransactions.jsx (+0 -1)
📝 packages/desktop-client/src/components/modals/ImportTransactions.test.js (+0 -0)
📝 packages/desktop-client/src/components/modals/MergeUnusedPayees.jsx (+1 -5)
📝 packages/desktop-client/src/components/modals/PlaidExternalMsg.tsx (+2 -2)
📝 packages/desktop-client/src/components/payees/PayeeTableRow.tsx (+4 -4)
📝 packages/desktop-client/src/components/reports/ChooseGraph.tsx (+2 -2)

...and 29 more files

📄 Description

  1. Upgrading prettier
  2. Enabling it for jsx files (it was not enabled before)
  3. Fixing all the issues (most are auto-fixed, but I'll call out the manual changes via inline comments)

🔄 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/actualbudget/actual/pull/2253 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 1/19/2024 **Status:** ✅ Merged **Merged:** 1/19/2024 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/prettier` --- ### 📝 Commits (2) - [`021e606`](https://github.com/actualbudget/actual/commit/021e6064e67f64a11f8dfdc4571b5c444b570e09) :arrow_up: (prettier) upgrade, fix issues and enable for jsx files - [`6a4eb14`](https://github.com/actualbudget/actual/commit/6a4eb147f506faf795bde3d8f8175385069afdb2) Use setupTests.js for tests ### 📊 Changes **49 files changed** (+362 additions, -295 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+2 -0) 📝 `package.json` (+4 -3) 📝 `packages/crdt/src/crdt/timestamp.ts` (+4 -4) 📝 `packages/desktop-client/src/components/Notifications.tsx` (+10 -10) 📝 `packages/desktop-client/src/components/Titlebar.tsx` (+10 -10) 📝 `packages/desktop-client/src/components/accounts/Balance.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/BudgetCategories.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/MobileBudgetTable.jsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/report/budgetsummary/Saved.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/util.ts` (+2 -2) 📝 `packages/desktop-client/src/components/common/Stack.tsx` (+15 -12) 📝 `packages/desktop-client/src/components/filters/FiltersMenu.jsx` (+6 -6) 📝 `packages/desktop-client/src/components/modals/GoCardlessExternalMsg.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/modals/ImportTransactions.jsx` (+0 -1) 📝 `packages/desktop-client/src/components/modals/ImportTransactions.test.js` (+0 -0) 📝 `packages/desktop-client/src/components/modals/MergeUnusedPayees.jsx` (+1 -5) 📝 `packages/desktop-client/src/components/modals/PlaidExternalMsg.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/payees/PayeeTableRow.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/reports/ChooseGraph.tsx` (+2 -2) _...and 29 more files_ </details> ### 📄 Description 1. Upgrading `prettier` 2. Enabling it for `jsx` files (it was not enabled before) 3. Fixing all the issues (most are auto-fixed, but I'll call out the manual changes via inline comments) --- <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 21:34:35 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#34400