Skip to content

Commit 2fafcea

Browse files
committed
Fixed Copilot review feedback
Updated the Source pnpm migration docs and scripts in response to Copilot review comments. The README now documents only the required global tools because Gulp resolves from the repo devDependency through pnpm scripts, and the ship script error message now spells uncommitted correctly.
1 parent b4afb3d commit 2fafcea

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ One neat trick is that you can also create custom one-off templates by adding th
2929

3030
# Development
3131

32-
Source styles are compiled using Gulp/PostCSS to polyfill future CSS spec. You'll need [Node](https://nodejs.org/), [pnpm](https://pnpm.io/) and [Gulp](https://gulpjs.com) installed globally. After that, from the theme's root directory:
32+
Source styles are compiled using Gulp/PostCSS to polyfill future CSS spec. You'll need [Node](https://nodejs.org/) and [pnpm](https://pnpm.io/). After that, from the theme's root directory:
3333

3434
```bash
3535
# install dependencies

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"pretest": "pnpm zip",
1818
"pretest:ci": "pnpm zip",
1919
"preship": "pnpm test",
20-
"ship": "STATUS=$(git status --porcelain); echo $STATUS; if [ -z \"$STATUS\" ]; then pnpm version && git push --follow-tags; else echo \"Uncomitted changes found.\" && exit 1; fi",
20+
"ship": "STATUS=$(git status --porcelain); echo $STATUS; if [ -z \"$STATUS\" ]; then pnpm version && git push --follow-tags; else echo \"Uncommitted changes found.\" && exit 1; fi",
2121
"postship": "git fetch && gulp release"
2222
},
2323
"packageManager": "pnpm@11.6.0",

0 commit comments

Comments
 (0)