[PR #6805] [MERGED] lint: enforce consistent TypeScript type import style #48568

Closed
opened 2026-04-26 10:29:04 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6805
Author: @MatissJanis
Created: 1/28/2026
Status: Merged
Merged: 2/10/2026
Merged by: @MatissJanis

Base: masterHead: matissjanis/enforce-consistent-type-imports


📝 Commits (2)

  • c61c2c2 Enforce consistent TypeScript type import style
  • 03bbd19 Merge branch 'master' into matissjanis/enforce-consistent-type-imports

📊 Changes

652 files changed (+1957 additions, -2248 deletions)

View changed files

📝 .oxlintrc.json (+2 -1)
📝 packages/api/methods.test.ts (+1 -1)
📝 packages/component-library/src/AlignedText.tsx (+1 -1)
📝 packages/component-library/src/Block.tsx (+2 -2)
📝 packages/component-library/src/Button.tsx (+2 -7)
📝 packages/component-library/src/Card.tsx (+2 -1)
📝 packages/component-library/src/ColorPicker.tsx (+5 -3)
📝 packages/component-library/src/FormError.tsx (+1 -1)
📝 packages/component-library/src/InitialFocus.ts (+1 -3)
📝 packages/component-library/src/InitialFocus.web.test.tsx (+2 -1)
📝 packages/component-library/src/InlineField.tsx (+2 -2)
📝 packages/component-library/src/Input.tsx (+6 -5)
📝 packages/component-library/src/Label.tsx (+2 -1)
📝 packages/component-library/src/Menu.tsx (+8 -10)
📝 packages/component-library/src/Paragraph.tsx (+2 -2)
📝 packages/component-library/src/Popover.tsx (+2 -1)
📝 packages/component-library/src/Select.tsx (+2 -1)
📝 packages/component-library/src/SpaceBetween.tsx (+3 -2)
📝 packages/component-library/src/Text.tsx (+3 -7)
📝 packages/component-library/src/TextOneLine.tsx (+1 -1)

...and 80 more files

📄 Description

Fixes https://github.com/actualbudget/actual/issues/6548

Only adding a new eslint rule for consistent import sorting and then running "yarn lint:fix" to auto-fix the problems.


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 27 14.47 MB 0%
loot-core 1 5.86 MB 0%
api 1 4.39 MB 0%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
27 14.47 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
static/js/index.js 9.33 MB 0%
static/js/indexeddb-main-thread-worker-e59fee74.js 12.94 kB 0%
static/js/workbox-window.prod.es5.js 5.64 kB 0%
static/js/da.js 106.62 kB 0%
static/js/de.js 178.39 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/en.js 164.55 kB 0%
static/js/es.js 173.83 kB 0%
static/js/fr.js 179.62 kB 0%
static/js/it.js 171.44 kB 0%
static/js/nb-NO.js 157.23 kB 0%
static/js/nl.js 106.65 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 154.57 kB 0%
static/js/sv.js 78.2 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 215.11 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.11 MB 0%
static/js/narrow.js 640.46 kB 0%
static/js/TransactionList.js 105.97 kB 0%
static/js/wide.js 160.07 kB 0%
static/js/AppliedFilters.js 9.71 kB 0%
static/js/usePayeeRuleCounts.js 11.79 kB 0%
static/js/useTransactionBatchActions.js 13.23 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.86 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
kcab.worker.G2jIa5TY.js 5.86 MB 0%

api

Total

Files count Total bundle size % Changed
1 4.39 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
bundle.api.js 4.39 MB 0%

🔄 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/6805 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 1/28/2026 **Status:** ✅ Merged **Merged:** 2/10/2026 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matissjanis/enforce-consistent-type-imports` --- ### 📝 Commits (2) - [`c61c2c2`](https://github.com/actualbudget/actual/commit/c61c2c229d819eadb3fc491245e93ea32ec4762d) Enforce consistent TypeScript type import style - [`03bbd19`](https://github.com/actualbudget/actual/commit/03bbd193f5e58e887a8935ab8124a23899b13bc5) Merge branch 'master' into matissjanis/enforce-consistent-type-imports ### 📊 Changes **652 files changed** (+1957 additions, -2248 deletions) <details> <summary>View changed files</summary> 📝 `.oxlintrc.json` (+2 -1) 📝 `packages/api/methods.test.ts` (+1 -1) 📝 `packages/component-library/src/AlignedText.tsx` (+1 -1) 📝 `packages/component-library/src/Block.tsx` (+2 -2) 📝 `packages/component-library/src/Button.tsx` (+2 -7) 📝 `packages/component-library/src/Card.tsx` (+2 -1) 📝 `packages/component-library/src/ColorPicker.tsx` (+5 -3) 📝 `packages/component-library/src/FormError.tsx` (+1 -1) 📝 `packages/component-library/src/InitialFocus.ts` (+1 -3) 📝 `packages/component-library/src/InitialFocus.web.test.tsx` (+2 -1) 📝 `packages/component-library/src/InlineField.tsx` (+2 -2) 📝 `packages/component-library/src/Input.tsx` (+6 -5) 📝 `packages/component-library/src/Label.tsx` (+2 -1) 📝 `packages/component-library/src/Menu.tsx` (+8 -10) 📝 `packages/component-library/src/Paragraph.tsx` (+2 -2) 📝 `packages/component-library/src/Popover.tsx` (+2 -1) 📝 `packages/component-library/src/Select.tsx` (+2 -1) 📝 `packages/component-library/src/SpaceBetween.tsx` (+3 -2) 📝 `packages/component-library/src/Text.tsx` (+3 -7) 📝 `packages/component-library/src/TextOneLine.tsx` (+1 -1) _...and 80 more files_ </details> ### 📄 Description Fixes https://github.com/actualbudget/actual/issues/6548 Only adding a new eslint rule for consistent import sorting and then running "yarn lint:fix" to auto-fix the problems. <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 27 | 14.47 MB | 0% loot-core | 1 | 5.86 MB | 0% api | 1 | 4.39 MB | 0% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 27 | 14.47 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/index.js | 9.33 MB | 0% static/js/indexeddb-main-thread-worker-e59fee74.js | 12.94 kB | 0% static/js/workbox-window.prod.es5.js | 5.64 kB | 0% static/js/da.js | 106.62 kB | 0% static/js/de.js | 178.39 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/en.js | 164.55 kB | 0% static/js/es.js | 173.83 kB | 0% static/js/fr.js | 179.62 kB | 0% static/js/it.js | 171.44 kB | 0% static/js/nb-NO.js | 157.23 kB | 0% static/js/nl.js | 106.65 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 154.57 kB | 0% static/js/sv.js | 78.2 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 215.11 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.11 MB | 0% static/js/narrow.js | 640.46 kB | 0% static/js/TransactionList.js | 105.97 kB | 0% static/js/wide.js | 160.07 kB | 0% static/js/AppliedFilters.js | 9.71 kB | 0% static/js/usePayeeRuleCounts.js | 11.79 kB | 0% static/js/useTransactionBatchActions.js | 13.23 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.86 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.G2jIa5TY.js | 5.86 MB | 0% </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.39 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- bundle.api.js | 4.39 MB | 0% </div> </details> </details> <!--- bundlestats-action-comment key:combined end ---> --- <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 10:29:04 -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#48568