[PR #1893] [MERGED] 🔧 (eslint) enabling 'react/no-unstable-nested-components' rule #4008

Closed
opened 2026-02-28 20:50:13 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1893
Author: @MatissJanis
Created: 11/10/2023
Status: Merged
Merged: 11/11/2023
Merged by: @MatissJanis

Base: masterHead: matiss/eslint-enable


📝 Commits (4)

  • d157f06 🔧 (eslint) enabling 'react/no-unstable-nested-components' rule
  • aa267f5 Release notes
  • e1e55b8 Fix name setterg
  • 20b3f3e Remove unnecessary eslint-ignore

📊 Changes

5 files changed (+136 additions, -121 deletions)

View changed files

📝 .eslintrc.js (+6 -0)
📝 packages/desktop-client/src/components/filters/SavedFilters.js (+123 -100)
📝 packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx (+1 -0)
📝 packages/desktop-client/src/components/transactions/MobileTransaction.js (+0 -21)
upcoming-release-notes/1893.md (+6 -0)

📄 Description

Enabled react/no-unstable-nested-components rule. This should help us build more maintainable code going forward. Also fixed some of the existing issues.


🔄 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/1893 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 11/10/2023 **Status:** ✅ Merged **Merged:** 11/11/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/eslint-enable` --- ### 📝 Commits (4) - [`d157f06`](https://github.com/actualbudget/actual/commit/d157f06805663808d6a3994e270a71b9f388dc3c) :wrench: (eslint) enabling 'react/no-unstable-nested-components' rule - [`aa267f5`](https://github.com/actualbudget/actual/commit/aa267f5689624278a339fb358f90c2d01627a79e) Release notes - [`e1e55b8`](https://github.com/actualbudget/actual/commit/e1e55b885a8ccca27664cd6d58fb934ab091e185) Fix name setterg - [`20b3f3e`](https://github.com/actualbudget/actual/commit/20b3f3ebfdd54fe2f94b6a07c284800f326ac301) Remove unnecessary eslint-ignore ### 📊 Changes **5 files changed** (+136 additions, -121 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+6 -0) 📝 `packages/desktop-client/src/components/filters/SavedFilters.js` (+123 -100) 📝 `packages/desktop-client/src/components/reports/graphs/NetWorthGraph.tsx` (+1 -0) 📝 `packages/desktop-client/src/components/transactions/MobileTransaction.js` (+0 -21) ➕ `upcoming-release-notes/1893.md` (+6 -0) </details> ### 📄 Description Enabled `react/no-unstable-nested-components` rule. This should help us build more maintainable code going forward. Also fixed some of the existing issues. --- <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-02-28 20:50:13 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#4008