[PR #746] [MERGED] Add prettier #1400

New issue

Closed

opened 2026-05-07 01:01:56 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 01:01:56 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/binwiederhier/ntfy/pull/746
Author: @nimbleghost
Created: 5/23/2023
Status: ✅ Merged
Merged: 5/24/2023
Merged by: @binwiederhier

Base: main ← Head: add-prettier

📝 Commits (4)

1761ec0 Move react-scripts to devDependencies
206ea31 Add prettier
6f6a2d1 Run prettier
a859ed9 Add .git-blame-ignore-revs

📊 Changes

54 files changed (+24625 additions, -6905 deletions)

View changed files

➕ .git-blame-ignore-revs (+5 -0)
📝 Makefile (+8 -1)
➕ web/.prettierignore (+2 -0)
📝 web/package-lock.json (+1671 -239)
📝 web/package.json (+7 -2)
📝 web/public/config.js (+20 -10)
📝 web/public/index.html (+55 -35)
📝 web/public/static/css/app.css (+5 -4)
📝 web/public/static/css/fonts.css (+16 -20)
📝 web/src/app/AccountApi.js (+395 -382)
📝 web/src/app/Api.js (+110 -103)
📝 web/src/app/Connection.js (+129 -91)
📝 web/src/app/ConnectionManager.js (+111 -91)
📝 web/src/app/Notifier.js (+84 -73)
📝 web/src/app/Poller.js (+47 -41)
📝 web/src/app/Prefs.js (+21 -21)
📝 web/src/app/Pruner.js (+26 -23)
📝 web/src/app/Session.js (+21 -21)
📝 web/src/app/SubscriptionManager.js (+187 -186)
📝 web/src/app/UserManager.js (+30 -30)

...and 34 more files

📄 Description

Adds prettier to npm, plus the relevant make targets.

Created a devDependencies sections for tooling, and moved react-scripts there too, which is the only other non-production dependency.

This PR needs to be rebased or merged with a merge commit into master, so that the .git-blame-ignore-refs file still references a valid commit.

_{🔄 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/binwiederhier/ntfy/pull/746 **Author:** [@nimbleghost](https://github.com/nimbleghost) **Created:** 5/23/2023 **Status:** ✅ Merged **Merged:** 5/24/2023 **Merged by:** [@binwiederhier](https://github.com/binwiederhier) **Base:** `main` ← **Head:** `add-prettier` --- ### 📝 Commits (4) - [`1761ec0`](https://github.com/binwiederhier/ntfy/commit/1761ec0207a392a62c97cf8d7eb24b1450d938fb) Move `react-scripts` to `devDependencies` - [`206ea31`](https://github.com/binwiederhier/ntfy/commit/206ea312bfadf6f73439673c1383dbfafca2b673) Add prettier - [`6f6a2d1`](https://github.com/binwiederhier/ntfy/commit/6f6a2d1f693070bf72e89d86748080e4825c9164) Run prettier - [`a859ed9`](https://github.com/binwiederhier/ntfy/commit/a859ed9f58ef9047f65050115404dd7efdbf60cd) Add .git-blame-ignore-revs ### 📊 Changes **54 files changed** (+24625 additions, -6905 deletions) <details> <summary>View changed files</summary> ➕ `.git-blame-ignore-revs` (+5 -0) 📝 `Makefile` (+8 -1) ➕ `web/.prettierignore` (+2 -0) 📝 `web/package-lock.json` (+1671 -239) 📝 `web/package.json` (+7 -2) 📝 `web/public/config.js` (+20 -10) 📝 `web/public/index.html` (+55 -35) 📝 `web/public/static/css/app.css` (+5 -4) 📝 `web/public/static/css/fonts.css` (+16 -20) 📝 `web/src/app/AccountApi.js` (+395 -382) 📝 `web/src/app/Api.js` (+110 -103) 📝 `web/src/app/Connection.js` (+129 -91) 📝 `web/src/app/ConnectionManager.js` (+111 -91) 📝 `web/src/app/Notifier.js` (+84 -73) 📝 `web/src/app/Poller.js` (+47 -41) 📝 `web/src/app/Prefs.js` (+21 -21) 📝 `web/src/app/Pruner.js` (+26 -23) 📝 `web/src/app/Session.js` (+21 -21) 📝 `web/src/app/SubscriptionManager.js` (+187 -186) 📝 `web/src/app/UserManager.js` (+30 -30) _...and 34 more files_ </details> ### 📄 Description Adds prettier to npm, plus the relevant make targets. Created a `devDependencies` sections for tooling, and moved react-scripts there too, which is the only other non-production dependency. This PR needs to be rebased or merged with a merge commit into master, so that the `.git-blame-ignore-refs` file still references a valid commit. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>