fix(build): add missing build/typecheck scripts and docs - #22
Conversation
davgonsan
left a comment
There was a problem hiding this comment.
Reviewed and ran it locally — this is a good, safe fix. bun run build was actually broken before this: the root script fanned out to build scripts in packages/shared and backend that didn't exist, so it always failed. This adds them (+ a typecheck alias) and excludes *.test.ts from tsc.
Verified on the branch:
bun run build→ clean (typechecks packages/shared + backend)bun run typecheck→ cleanbun test→ 31 pass / 0 fail
One ask: please retarget this to develop, not main. Active development is on develop (that's where the app + backend work is landing), so a build/tooling fix should go there first — otherwise develop keeps the broken bun run build until a main → develop merge catches up. If the intent is main-first, it needs a prompt forward-merge into develop.
LGTM once retargeted.
e053d50 to
921a5a0
Compare
|
Thanks! Retargeted to |
|
@davgonsan Me gustaría poder seguir aportando al proyecto pero veo que estos últimos días ha parado mucho la actividad y hasta tener estas dos primeras PR no he querido seguir desarrollando, es posible estar en algún grupo o chat de trabajo con ustedes para estar al tanto de como va avanzando el proyecto? |
¿Qué hace este PR?
Corrige
bun run buildy los comandos relacionados.Añade scripts de typecheck en
packages/sharedybackend, y ajusta la documentación para que los comandos sean consistentes.¿Por qué?
bun run buildestaba roto: el script raíz llamababuildenpackages/sharedybackend, pero esos paquetes no tenían ese script.Además,
packages/sharedestaba typecheckeando también tests, y fallaba por errores de tipos debun:test.¿Cómo probarlo?
bun run buildbun run typecheckbun testChecklist
bun test)/ingestni firma Ed25519