Skip to content

Commit 8eaa0b6

Browse files
committed
Fixup some nits in the CONTRIBUTING instructions.
1 parent 01de86a commit 8eaa0b6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ This extension ***must never transmit any data outside the browser***.
1919
### Hotreload development
2020

2121
- `pnpm install`
22-
- `pnpm run dev`
22+
- `pnpm dev`
2323
- open [`chrome://extensions`](chrome://extensions)
2424
- toggle **Developer mode** (top-right)
2525
- click "Load unpacked" (far left)
@@ -45,7 +45,7 @@ This is a [WXT](https://wxt.dev/)-based browser extension that
4545

4646
- [`src/entrypoints/content.ts`](src/entrypoints/content.ts) - injected into every webpage
4747
- [`src/entrypoints/background.ts`](src/entrypoints/background.ts) - service worker that manages state and handles messages
48-
- [`src/entrypoints/popup/popup.tsx`](src/entrypoints/popup/popup.tsx) - popup (html/css/tsx) with shadcn/ui table components
48+
- [`src/entrypoints/popup/popup.tsx`](src/entrypoints/popup/popup.tsx) - popup (react html + TailwindCSS)
4949

5050
```mermaid
5151
graph TD
@@ -94,7 +94,7 @@ We maintain a corpus of test pages in two formats for testing the browser extens
9494
- `npx playwright codegen https://github.com/login --save-storage=playwright/.auth/gh.json` will store new auth tokens
9595
- login manually, then close the browser
9696
- ***these cookies are very sensitive! we only run this script using a test account that has no permissions or memberships to anything, recommend you do the same!***
97-
- `pnpm run corpus:har:record` records new HAR files using those auth tokens (it needs args, run it with no args for docs)
97+
- `pnpm corpus:har:record` records new HAR files using those auth tokens (it needs args, run it with no args for docs)
9898
- DO NOT COMMIT AND PUSH NEW OR CHANGED HAR files!
9999
- we try to sanitize these (see `corpus-har-record.ts` for details) but there may be important PII in them
100100
- if you need new HAR files for something, let us know and we will generate them ourselves using a dummy account

0 commit comments

Comments
 (0)