[PR #934] [MERGED] Use Peggy instead of deprecated Peg.js #3417

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/934
Author: @sinistersnare
Created: 4/22/2023
Status: Merged
Merged: 4/23/2023
Merged by: @j-f1

Base: masterHead: feature/peggy


📝 Commits (1)

  • 422b566 Use Peggy instead of deprecated Peg.js

📊 Changes

9 files changed (+69 additions, -81 deletions)

View changed files

📝 packages/loot-core/jest.config.js (+1 -1)
📝 packages/loot-core/jest.web.config.js (+1 -1)
📝 packages/loot-core/package.json (+1 -3)
packages/loot-core/peg-loader.js (+9 -0)
packages/loot-core/peg-transform.mjs (+24 -0)
📝 packages/loot-core/src/server/budget/goal-template.pegjs (+1 -1)
📝 packages/loot-core/webpack/webpack.browser.config.js (+1 -6)
upcoming-release-notes/934.md (+6 -0)
📝 yarn.lock (+25 -69)

📄 Description

Hi there, Peg.js is unmaintained, so I figure you all would appreciate if I replaced it with the drop-in replacement of Peggy. This is work I am breaking out of #918.

Peggy adds new features like source map support that we could use, although I do not include that in this change-set. It may be useful for debugging changes to the .pegjs file we have.e


🔄 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/934 **Author:** [@sinistersnare](https://github.com/sinistersnare) **Created:** 4/22/2023 **Status:** ✅ Merged **Merged:** 4/23/2023 **Merged by:** [@j-f1](https://github.com/j-f1) **Base:** `master` ← **Head:** `feature/peggy` --- ### 📝 Commits (1) - [`422b566`](https://github.com/actualbudget/actual/commit/422b56652b3209b789a110691aa3452159866d06) Use Peggy instead of deprecated Peg.js ### 📊 Changes **9 files changed** (+69 additions, -81 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/jest.config.js` (+1 -1) 📝 `packages/loot-core/jest.web.config.js` (+1 -1) 📝 `packages/loot-core/package.json` (+1 -3) ➕ `packages/loot-core/peg-loader.js` (+9 -0) ➕ `packages/loot-core/peg-transform.mjs` (+24 -0) 📝 `packages/loot-core/src/server/budget/goal-template.pegjs` (+1 -1) 📝 `packages/loot-core/webpack/webpack.browser.config.js` (+1 -6) ➕ `upcoming-release-notes/934.md` (+6 -0) 📝 `yarn.lock` (+25 -69) </details> ### 📄 Description Hi there, `Peg.js` is unmaintained, so I figure you all would appreciate if I replaced it with the drop-in replacement of Peggy. This is work I am breaking out of #918. Peggy adds new features like source map support that we could use, although I do not include that in this change-set. It may be useful for debugging changes to the .pegjs file we have.e --- <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:41:21 -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#3417