Rikki Schulte

1. you need `files` entry for `dist/src`. only `main` is included by default. `dist/src/cli` won't ship without it. 2. `prepublishOnly` should be `prepublish` to hook into default `npm publish` behavior

still getting: `warning "[email protected]" has no binaries`

this package has a binary that works, if folks are looking: `@dylanvann/flow-to-typescript`

I think I’ll mark this as an enhancement because it should do this for a11y and other reasons, but it did not before. I think we have a global keybindings...

@thomasheyenbrock meet @connorshea who contributed the headers pane amongst other bugfixes! @harshithpabbati was helping with these efforts as well iirc

@rxliuli ah, the email used for CLA has to be the same email that’s used in each commit. Also I have to approve the action workflows for the first PR...

It looks like we need to use a base image that already has pnpm ideally, or we can use an action to manually install pnpm for each workflow

@rxliuli awesome! Also, weird, not sure why i had to re-approve the actions

@rxliuli something is off with lint staged, so for now you need to run pnpm format.

I can’t remember why i migrated the actions to re-install modules from the shared cache individually rather than having a shared action for npm install and build. It started with...