cursorless icon indicating copy to clipboard operation
cursorless copied to clipboard

Revisit `compile` / `build` npm script names / terminology

Open pokey opened this issue 1 year ago • 0 comments

We're not very consistent with our terminology around compile and build in the various packages of our monorepo. We should

  • [ ] Look at compile / build in each of our packages, as well as the root package, to determine what they're doing
  • [ ] Come up with consistent terminology. I'd be tempted to remove compile entirely, and instead have build / build:foo for anything we're building and lint:tsc to replace the places where we use the term compile for type checking today. Because we don't actually emit anything from typescript, it's a bit of a misnomer to call it compile
  • [ ] Revisit whether the contributor really needs to run pnpm compile during initial install https://github.com/cursorless-dev/cursorless/blob/1183dd3cd4991db871536599252ec782293a1512/docs/contributing/CONTRIBUTING.md#L23
  • [ ] Investigate impact on CI

pokey avatar Jun 06 '24 09:06 pokey