[PR #2111] [MERGED] ♻️ (eslint) disallow unnecessary curly braces #45942

Closed
opened 2026-04-26 08:03:03 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2111
Author: @MatissJanis
Created: 12/21/2023
Status: Merged
Merged: 12/22/2023
Merged by: @MatissJanis

Base: masterHead: matiss/eslint-curly-braces


📝 Commits (2)

  • 570f6c3 ♻️ (eslint) disallow unneccessary curly braces
  • 06ea1cc Release notes

📊 Changes

8 files changed (+16 additions, -9 deletions)

View changed files

📝 .eslintrc.js (+1 -0)
📝 packages/desktop-client/src/components/filters/FiltersButton.tsx (+1 -1)
📝 packages/desktop-client/src/components/reports/ReportTableHeader.tsx (+4 -4)
📝 packages/desktop-client/src/components/reports/ReportTableTotals.tsx (+1 -1)
📝 packages/desktop-client/src/components/reports/SaveReport.tsx (+1 -1)
📝 packages/desktop-client/src/components/reports/graphs/BarGraph.tsx (+1 -1)
📝 packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx (+1 -1)
upcoming-release-notes/2111.md (+6 -0)

📄 Description

Disallow using unnecessary curly braces.

Should make our codebase slightly more consistent.


🔄 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/2111 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 12/21/2023 **Status:** ✅ Merged **Merged:** 12/22/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/eslint-curly-braces` --- ### 📝 Commits (2) - [`570f6c3`](https://github.com/actualbudget/actual/commit/570f6c3cb33cd7c85b1f354e1bd361431398ec8c) :recycle: (eslint) disallow unneccessary curly braces - [`06ea1cc`](https://github.com/actualbudget/actual/commit/06ea1ccb063e95181725eb12d39192e6014c471a) Release notes ### 📊 Changes **8 files changed** (+16 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+1 -0) 📝 `packages/desktop-client/src/components/filters/FiltersButton.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/reports/ReportTableHeader.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/reports/ReportTableTotals.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/reports/SaveReport.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/reports/graphs/BarGraph.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/reports/graphs/StackedBarGraph.tsx` (+1 -1) ➕ `upcoming-release-notes/2111.md` (+6 -0) </details> ### 📄 Description Disallow using unnecessary curly braces. Should make our codebase slightly more consistent. --- <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-26 08:03:03 -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#45942