[PR #733] [MERGED] Remove outdated part of the postinstall script #65655

Closed
opened 2026-05-13 23:39:49 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/733
Author: @j-f1
Created: 3/9/2023
Status: Merged
Merged: 3/9/2023
Merged by: @j-f1

Base: masterHead: jed/rm-rm


📝 Commits (1)

  • 61d92ee Remove outdated part of postinstall script

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 package.json (+1 -1)

📄 Description

Yarn v2 does not put a node_modules folder inside of packages/loot-design so this cannot possibly do anything. Additionally, rm is not available in cmd.exe for Windows. Ref: #730.


🔄 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/733 **Author:** [@j-f1](https://github.com/j-f1) **Created:** 3/9/2023 **Status:** ✅ Merged **Merged:** 3/9/2023 **Merged by:** [@j-f1](https://github.com/j-f1) **Base:** `master` ← **Head:** `jed/rm-rm` --- ### 📝 Commits (1) - [`61d92ee`](https://github.com/actualbudget/actual/commit/61d92eebe701df102a2656cf3f9f599b789cd8e1) Remove outdated part of postinstall script ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `package.json` (+1 -1) </details> ### 📄 Description Yarn v2 does not put a `node_modules` folder inside of `packages/loot-design` so this cannot possibly do anything. Additionally, `rm` is not available in `cmd.exe` for Windows. Ref: #730. --- <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-05-13 23:39:49 -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#65655