[PR #1599] [MERGED] 🔧 (eslint) convert rules from ERROR to WARN to improve devX #3840

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1599
Author: @MatissJanis
Created: 8/26/2023
Status: Merged
Merged: 9/9/2023
Merged by: @MatissJanis

Base: masterHead: matiss/eslint-warn


📝 Commits (6)

  • 18711e3 🔧 (eslint) convert rules from ERROR to WARN to improve devX
  • 0dbbb08 Release notes
  • a960270 Rename 1598.md to 1599.md
  • a1a61e5 Merge branch 'master' into matiss/eslint-warn
  • dcd6284 Merge branch 'master' into matiss/eslint-warn
  • a85bba3 Merge branch 'master' into matiss/eslint-warn

📊 Changes

2 files changed (+26 additions, -20 deletions)

View changed files

📝 .eslintrc.js (+20 -20)
upcoming-release-notes/1599.md (+6 -0)

📄 Description

Convert eslint rules from "error" to "warn" to improve dev-experience; CI jobs treat warnings as errors, so we still have the same level of protection, but the local-dev experience is slightly improved


🔄 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/1599 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 8/26/2023 **Status:** ✅ Merged **Merged:** 9/9/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/eslint-warn` --- ### 📝 Commits (6) - [`18711e3`](https://github.com/actualbudget/actual/commit/18711e3218dba50828c3ed1f03435a12ad7b1c4c) :wrench: (eslint) convert rules from ERROR to WARN to improve devX - [`0dbbb08`](https://github.com/actualbudget/actual/commit/0dbbb08fc4ff76516f7e3a749030c30da5ebb581) Release notes - [`a960270`](https://github.com/actualbudget/actual/commit/a9602704e23afcc51542bed85c8c976386dbc96e) Rename 1598.md to 1599.md - [`a1a61e5`](https://github.com/actualbudget/actual/commit/a1a61e5ee261f83bb45db8bb07fe70575b637ba1) Merge branch 'master' into matiss/eslint-warn - [`dcd6284`](https://github.com/actualbudget/actual/commit/dcd6284894a44ac166fef893f702afc2e55592b4) Merge branch 'master' into matiss/eslint-warn - [`a85bba3`](https://github.com/actualbudget/actual/commit/a85bba3464a3083dc45876333b49586d5d6cba94) Merge branch 'master' into matiss/eslint-warn ### 📊 Changes **2 files changed** (+26 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+20 -20) ➕ `upcoming-release-notes/1599.md` (+6 -0) </details> ### 📄 Description Convert eslint rules from "error" to "warn" to improve dev-experience; CI jobs treat warnings as errors, so we still have the same level of protection, but the local-dev experience is slightly improved --- <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:47:35 -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#3840