Skip to content

Commit 9616f44

Browse files
committed
docs: update REVIEW.md — add PR #32 (CI type-check fixes)
1 parent 827c044 commit 9616f44

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

REVIEW.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -266,6 +266,7 @@ app/
266266
- ✅ Reduce npm vulnerabilities from 60 → 34 (PRs #29, #30)
267267
- ✅ Document ecosystem alignment (all myst projects on Remix v1)
268268
- ✅ Fix bugs, add CI pipeline, Node engine update, CONTRIBUTING.md — PR [#31](https://github.com/QuantEcon/quantecon-theme-src/pull/31)
269+
- ✅ Fix pre-existing type-check errors caught by new CI — PR [#32](https://github.com/QuantEcon/quantecon-theme-src/pull/32)
269270

270271
---
271272

@@ -352,6 +353,7 @@ As part of this review (February 2026), all open PRs were triaged and resolved:
352353
| [#29](https://github.com/QuantEcon/quantecon-theme-src/pull/29) | Fix CI: bump @remix-run/* 1.17→1.19, security fixes | Fixed ERESOLVE CI failure, reduced vulns 60→37, added npm overrides |
353354
| [#30](https://github.com/QuantEcon/quantecon-theme-src/pull/30) | Upgrade @myst-theme/* 0.14.x→1.1.2 | Major dep upgrade, aligned with upstream, reduced vulns 37→34 |
354355
| [#31](https://github.com/QuantEcon/quantecon-theme-src/pull/31) | Quick wins: bug fixes, CI, Node engine, CONTRIBUTING.md | Fix 3 bugs, add CI pipeline, .nvmrc, contributor guide |
356+
| [#32](https://github.com/QuantEcon/quantecon-theme-src/pull/32) | Fix CI type-check errors (lodash types, Buffer→Uint8Array) | Resolve 3 pre-existing type errors caught by new CI pipeline |
355357

356358
### Issues Opened
357359

0 commit comments

Comments
 (0)