[PR #213] [MERGED] Enforce linting in desktop-client #3058

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/213
Author: @TomAFrench
Created: 8/25/2022
Status: Merged
Merged: 8/25/2022
Merged by: @TomAFrench

Base: masterHead: lint-desktop-client


📝 Commits (4)

  • 3ee7ae1 style: fix or silence linting errors in desktop-client
  • 4f0bfe4 ci: lint desktop-client in CI
  • 677faa6 build: make eslint-plugin-prettier a dependency of desktop-client
  • 1d0b272 style: prettify .eslintrc.js

📊 Changes

6 files changed (+25 additions, -9 deletions)

View changed files

📝 .eslintrc.js (+6 -6)
📝 package.json (+1 -1)
📝 packages/desktop-client/package.json (+3 -1)
📝 packages/desktop-client/src/components/accounts/TransactionsTable.test.js (+10 -1)
📝 packages/desktop-client/src/components/budget/budget-sheets-old.js (+4 -0)
📝 yarn.lock (+1 -0)

📄 Description

I've fixed some of the more obvious errors and silenced the rest for now (especially as it seems like that file can just be deleted due to it being old?).


🔄 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/213 **Author:** [@TomAFrench](https://github.com/TomAFrench) **Created:** 8/25/2022 **Status:** ✅ Merged **Merged:** 8/25/2022 **Merged by:** [@TomAFrench](https://github.com/TomAFrench) **Base:** `master` ← **Head:** `lint-desktop-client` --- ### 📝 Commits (4) - [`3ee7ae1`](https://github.com/actualbudget/actual/commit/3ee7ae16cc7a6b7a60bed4ac75f93e05b988d42e) style: fix or silence linting errors in desktop-client - [`4f0bfe4`](https://github.com/actualbudget/actual/commit/4f0bfe4a1f4a0a4ec830a2b401e8b341a22f4450) ci: lint desktop-client in CI - [`677faa6`](https://github.com/actualbudget/actual/commit/677faa611d4de7df826be35795fd665a705624dc) build: make eslint-plugin-prettier a dependency of desktop-client - [`1d0b272`](https://github.com/actualbudget/actual/commit/1d0b2721dbaa8c4050040e85b222c087e45ef437) style: prettify .eslintrc.js ### 📊 Changes **6 files changed** (+25 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `.eslintrc.js` (+6 -6) 📝 `package.json` (+1 -1) 📝 `packages/desktop-client/package.json` (+3 -1) 📝 `packages/desktop-client/src/components/accounts/TransactionsTable.test.js` (+10 -1) 📝 `packages/desktop-client/src/components/budget/budget-sheets-old.js` (+4 -0) 📝 `yarn.lock` (+1 -0) </details> ### 📄 Description I've fixed some of the more obvious errors and silenced the rest for now (especially as it seems like that file can just be deleted due to it being old?). --- <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:35:43 -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#3058