[PR #6464] [MERGED] lint: add react/jsx-boolean-value lint rule #32471

Closed
opened 2026-04-18 08:30:10 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6464
Author: @MatissJanis
Created: 12/21/2025
Status: ✅ Merged
Merged: 12/21/2025
Merged by: @MatissJanis

Base: master ← Head: matiss/add-jsx-boolean-value-lint-rule


📝 Commits (1)

  • a167406 lint: add react/jsx-boolean-value lint rule

📊 Changes

78 files changed (+121 additions, -114 deletions)

View changed files

📝 .oxlintrc.json (+1 -0)
📝 packages/component-library/src/Popover.tsx (+1 -1)
📝 packages/desktop-client/src/components/FatalError.tsx (+1 -1)
📝 packages/desktop-client/src/components/FinancesApp.tsx (+1 -1)
📝 packages/desktop-client/src/components/accounts/Balance.tsx (+1 -1)
📝 packages/desktop-client/src/components/accounts/BalanceHistoryGraph.tsx (+1 -1)
📝 packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx (+1 -1)
📝 packages/desktop-client/src/components/admin/UserDirectory/UserDirectoryHeader.tsx (+1 -1)
📝 packages/desktop-client/src/components/admin/UserDirectory/UserDirectoryRow.tsx (+4 -4)
📝 packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/autocomplete/Autocomplete.tsx (+1 -1)
📝 packages/desktop-client/src/components/autocomplete/CategoryAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/autocomplete/FilterAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/autocomplete/PayeeAutocomplete.tsx (+1 -1)
📝 packages/desktop-client/src/components/autocomplete/ReportAutocomplete.tsx (+2 -2)
📝 packages/desktop-client/src/components/banksync/AccountRow.tsx (+1 -1)
📝 packages/desktop-client/src/components/banksync/FieldMapping.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/BudgetCategories.tsx (+2 -2)
📝 packages/desktop-client/src/components/budget/ExpenseCategory.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/ExpenseGroup.tsx (+1 -1)

...and 58 more files

📄 Description

Tiny pet peeve.. added react/jsx-boolean-value rule and auto-fixed the violations.

Summary by CodeRabbit

  • Chores

    • Added new linting rule to enforce consistent JSX boolean attribute syntax.
  • Style

    • Refactored components across the application to standardize boolean prop formatting.

✏️ Tip: You can customize this high-level summary in your review settings.


🔄 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/6464 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 12/21/2025 **Status:** ✅ Merged **Merged:** 12/21/2025 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/add-jsx-boolean-value-lint-rule` --- ### 📝 Commits (1) - [`a167406`](https://github.com/actualbudget/actual/commit/a167406d8c33ffe717f0f652bb183ac6275d687c) lint: add react/jsx-boolean-value lint rule ### 📊 Changes **78 files changed** (+121 additions, -114 deletions) <details> <summary>View changed files</summary> 📝 `.oxlintrc.json` (+1 -0) 📝 `packages/component-library/src/Popover.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/FatalError.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/FinancesApp.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/accounts/Balance.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/accounts/BalanceHistoryGraph.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/admin/UserAccess/UserAccessRow.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/admin/UserDirectory/UserDirectoryHeader.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/admin/UserDirectory/UserDirectoryRow.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/autocomplete/Autocomplete.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/autocomplete/CategoryAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/autocomplete/FilterAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/autocomplete/PayeeAutocomplete.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/autocomplete/ReportAutocomplete.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/banksync/AccountRow.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/banksync/FieldMapping.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/BudgetCategories.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/budget/ExpenseCategory.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/ExpenseGroup.tsx` (+1 -1) _...and 58 more files_ </details> ### 📄 Description Tiny pet peeve.. added `react/jsx-boolean-value` rule and auto-fixed the violations. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Added new linting rule to enforce consistent JSX boolean attribute syntax. * **Style** * Refactored components across the application to standardize boolean prop formatting. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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-18 08:30:10 -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#32471