chore: Fix markdown lint errors and add upgrade @aklinker1/check to
include markdownlint
This commit is contained in:
@@ -23,9 +23,6 @@ jobs:
|
||||
- name: Basic Checks
|
||||
run: pnpm check
|
||||
|
||||
- name: Markdownlint
|
||||
run: pnpm check:md
|
||||
|
||||
builds:
|
||||
name: Builds
|
||||
runs-on: ubuntu-22.04
|
||||
|
||||
@@ -54,8 +54,10 @@ See <https://github.com/wxt-dev/wxt/issues/784>
|
||||
To upgrade, you have two options:
|
||||
|
||||
1. **Stop using the polyfill**
|
||||
|
||||
- If you're already using `extensionApi: "chrome"`, then you're not using the polyfill and there is nothing to change!
|
||||
- Otherwise there is only one change: `browser.runtime.onMessage` no longer supports using promises to return a response:
|
||||
|
||||
```ts
|
||||
browser.runtime.onMessage.addListener(async () => { // [!code --]
|
||||
const res = await someAsyncWork(); // [!code --]
|
||||
@@ -67,6 +69,7 @@ To upgrade, you have two options:
|
||||
return true; // [!code ++]
|
||||
});
|
||||
```
|
||||
|
||||
2. **Continue using the polyfill** - If you want to keep using the polyfill, you can! One less thing to worry about during this upgrade.
|
||||
|
||||
- Install `webextension-polyfill` and WXT's [new polyfill module](https://www.npmjs.com/package/@wxt-dev/webextension-polyfill):
|
||||
|
||||
+1
-2
@@ -7,7 +7,6 @@
|
||||
"packageManager": "pnpm@10.10.0",
|
||||
"scripts": {
|
||||
"check": "check && pnpm -r --sequential run check",
|
||||
"check:md": "markdownlint .",
|
||||
"test": "pnpm -r --sequential run test run",
|
||||
"test:coverage": "pnpm -r --sequential run test:coverage",
|
||||
"prepare": "simple-git-hooks",
|
||||
@@ -19,7 +18,7 @@
|
||||
},
|
||||
"devDependencies": {
|
||||
"@aklinker1/buildc": "^1.1.4",
|
||||
"@aklinker1/check": "2.0.0",
|
||||
"@aklinker1/check": "^2.1.0",
|
||||
"@commitlint/config-conventional": "^19.8.0",
|
||||
"@commitlint/types": "^19.8.0",
|
||||
"@types/fs-extra": "^11.0.4",
|
||||
|
||||
Generated
+12
-2
@@ -17,8 +17,8 @@ importers:
|
||||
specifier: ^1.1.4
|
||||
version: 1.1.4
|
||||
'@aklinker1/check':
|
||||
specifier: 2.0.0
|
||||
version: 2.0.0
|
||||
specifier: ^2.1.0
|
||||
version: 2.1.0
|
||||
'@commitlint/config-conventional':
|
||||
specifier: ^19.8.0
|
||||
version: 19.8.0
|
||||
@@ -693,6 +693,10 @@ packages:
|
||||
resolution: {integrity: sha512-itoIc3WlLCtoVEwTIi/f6E4UKXJQJhYyMzbBqbQ73uvIsChgXKfmZHjaUwz4PyBtmkvlO4yhkh1JSQUzy2x5jA==}
|
||||
hasBin: true
|
||||
|
||||
'@aklinker1/check@2.1.0':
|
||||
resolution: {integrity: sha512-x+0vxb0vHV+6ZskcLYobI6FCAFG1jwVGgKuH9pTfHLW3vlzbGSGUKcxbCuMI3Q51WZS8oE4UfB+4v5M2+0uz1g==}
|
||||
hasBin: true
|
||||
|
||||
'@aklinker1/rollup-plugin-visualizer@5.12.0':
|
||||
resolution: {integrity: sha512-X24LvEGw6UFmy0lpGJDmXsMyBD58XmX1bbwsaMLhNoM+UMQfQ3b2RtC+nz4b/NoRK5r6QJSKJHBNVeUdwqybaQ==}
|
||||
engines: {node: '>=14'}
|
||||
@@ -5245,6 +5249,12 @@ snapshots:
|
||||
ci-info: 4.2.0
|
||||
citty: 0.1.6
|
||||
|
||||
'@aklinker1/check@2.1.0':
|
||||
dependencies:
|
||||
'@antfu/utils': 0.7.10
|
||||
ci-info: 4.2.0
|
||||
citty: 0.1.6
|
||||
|
||||
'@aklinker1/rollup-plugin-visualizer@5.12.0(rollup@4.34.9)':
|
||||
dependencies:
|
||||
open: 8.4.2
|
||||
|
||||
Reference in New Issue
Block a user