[PR #6449] [MERGED] lint: move eslint-native rules to oxlint #20896

Closed
opened 2026-04-14 21:48:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

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

Base: masterHead: matiss/linting-config-updates


📝 Commits (2)

  • b245c0f Update linting configuration and fix related issues
  • 9c289bc Fix type coercion for comment ID in check-first-comment.js to ensure proper comparison with summary comment IDs

📊 Changes

8 files changed (+80 additions, -128 deletions)

View changed files

📝 .github/actions/ai-generated-release-notes/check-first-comment.js (+2 -2)
📝 .oxlintrc.json (+66 -0)
📝 eslint.config.mjs (+1 -121)
📝 packages/crdt/src/crdt/merkle.ts (+1 -1)
📝 packages/loot-core/src/platform/server/sqlite/unicodeLike.test.ts (+1 -0)
📝 packages/loot-core/src/server/cloud-storage.ts (+1 -1)
📝 packages/loot-core/src/server/sync/migrate.test.ts (+2 -3)
upcoming-release-notes/6449.md (+6 -0)

📄 Description

Moving almost all native eslint rules over to oxlint. A few are not available in oxlint and hence I have kept them in eslint for now.


🔄 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/6449 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 12/19/2025 **Status:** ✅ Merged **Merged:** 12/19/2025 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/linting-config-updates` --- ### 📝 Commits (2) - [`b245c0f`](https://github.com/actualbudget/actual/commit/b245c0f33462d1a841b08f185d6fffe78d116570) Update linting configuration and fix related issues - [`9c289bc`](https://github.com/actualbudget/actual/commit/9c289bcf5ecc59ad23133c020962278f35c9f1f5) Fix type coercion for comment ID in check-first-comment.js to ensure proper comparison with summary comment IDs ### 📊 Changes **8 files changed** (+80 additions, -128 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/ai-generated-release-notes/check-first-comment.js` (+2 -2) 📝 `.oxlintrc.json` (+66 -0) 📝 `eslint.config.mjs` (+1 -121) 📝 `packages/crdt/src/crdt/merkle.ts` (+1 -1) 📝 `packages/loot-core/src/platform/server/sqlite/unicodeLike.test.ts` (+1 -0) 📝 `packages/loot-core/src/server/cloud-storage.ts` (+1 -1) 📝 `packages/loot-core/src/server/sync/migrate.test.ts` (+2 -3) ➕ `upcoming-release-notes/6449.md` (+6 -0) </details> ### 📄 Description Moving almost all native eslint rules over to oxlint. A few are not available in oxlint and hence I have kept them in eslint for now. --- <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-14 21:48:45 -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#20896