update to pnpm6 and minor setup improvements (#30)

* use changelog-github formatter

* update to pnpm 6

* require node version with esmodule support

* use engine-strict to ensure node version

* chore: use toHaveLength to fix lint warning

Author: dominikg

.changeset/config.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://unpkg.com/@changesets/[email protected]/schema.json",
-  "changelog": "@changesets/cli/changelog",
+  "changelog": "@changesets/changelog-github",
   "commit": false,
   "linked": [],
   "access": "public",

.github/workflows/ci.yml

@@ -31,7 +31,7 @@ jobs:
         uses: actions/checkout@v2
 
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: pnpm-store
@@ -82,7 +82,7 @@ jobs:
           path: ${{ env.pnpm_store_path }}
           key: ${{ matrix.os }}-node-v${{ matrix.node }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: install
@@ -117,7 +117,7 @@ jobs:
           path: ${{ env.pnpm_store_path }}
           key: ${{ matrix.os }}-node-v${{ matrix.node }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: install
@@ -150,7 +150,7 @@ jobs:
           key: ${{ matrix.os }}-node-v${{ matrix.node }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
 
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: install
@@ -209,7 +209,7 @@ jobs:
           path: ${{ env.pnpm_store_path }}
           key: ${{ matrix.os }}-node-v${{ matrix.node }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: install

.github/workflows/release.yml

@@ -26,7 +26,7 @@ jobs:
           node-version: 14
       # install pnpm and try to reuse cache from ci action by using same cache keys
       - name: install pnpm
-        run: npm i -g pnpm@5
+        run: npm i -g pnpm@6
       - name: set pnpm store-dir
         run: pnpm config set store-dir ${{ env.pnpm_store_path }}
       - name: pnpm-store

.npmrc

@@ -0,0 +1 @@
+engine-strict=true

pnpmfile.js renamed to .pnpmfile.cjs

@@ -24,9 +24,10 @@
     "format:fix": "pnpm format -- --write",
     "fixup": "run-s lint:fix format:fix",
     "update-deps": "ncu -u",
-    "release": "pnpm run build && pnpx changeset publish"
+    "release": "pnpm run build && pnpx --no changeset publish"
   },
   "devDependencies": {
+    "@changesets/changelog-github": "^0.3.0",
     "@changesets/cli": "^2.15.0",
     "@microsoft/api-extractor": "^7.13.2",
     "@types/fs-extra": "^9.0.10",
@@ -59,7 +60,7 @@
     "npm-check-updates": "^11.3.0",
     "npm-run-all": "^4.1.5",
     "playwright-core": "^1.10.0",
-    "pnpm": "^5.18.9",
+    "pnpm": "^6.0.1",
     "prettier": "^2.2.1",
     "prettier-plugin-svelte": "^2.2.0",
     "rimraf": "^3.0.2",
@@ -87,8 +88,9 @@
     ]
   },
   "engines": {
-    "pnpm": ">=5",
+    "pnpm": ">=6",
     "yarn": "forbidden, use pnpm",
-    "npm": "forbidden, use pnpm"
+    "npm": "forbidden, use pnpm",
+    "node": ">=12.17.0"
   }
 }

package.json

@@ -84,7 +84,7 @@ describe('kit-node', () => {
 
 				it('should render additional child components', async () => {
 					let buttons = await page.$$('button');
-					expect(buttons.length).toBe(1);
+					expect(buttons).toHaveLength(1);
 					expect(await getText(buttons[0])).toBe('Clicks: 0');
 					await updateIndexSvelte((content) =>
 						content.replace(
@@ -93,7 +93,7 @@ describe('kit-node', () => {
 						)
 					);
 					buttons = await page.$$('button');
-					expect(buttons.length).toBe(2);
+					expect(buttons).toHaveLength(2);
 					expect(await getText(buttons[0])).toBe('Clicks: 0');
 					expect(await getText(buttons[1])).toBe('Clicks: 0');
 					await buttons[1].click();
@@ -103,7 +103,7 @@ describe('kit-node', () => {
 						content.replace('<Counter id="hmr-test-counter"/>\n', '')
 					);
 					buttons = await page.$$('button');
-					expect(buttons.length).toBe(1);
+					expect(buttons).toHaveLength(1);
 					expect(await getText(buttons[0])).toBe('Clicks: 0');
 				});