Skip to content

Bazel js toolchain rewrite#2206

Open
ikholopov-omni wants to merge 11 commits into
dataform-co:mainfrom
ikholopov-omni:bazel7js
Open

Bazel js toolchain rewrite#2206
ikholopov-omni wants to merge 11 commits into
dataform-co:mainfrom
ikholopov-omni:bazel7js

Conversation

@ikholopov-omni

@ikholopov-omni ikholopov-omni commented Jun 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Migrate to maintained aspect-rules for JS toolchain
  • Use pnpm instead of yarn, remove no-longer required scripts for switching registry (as pnpm-lock doesn't hard-code the domain, it always respects .npmrc config)
  • Keep the same bundling layout - ESM for CLI, CommonJS for Core
  • Updated scripts to use updated tooling
  • Complete migration to BzlMod, cleanup WORKSPACE

@ikholopov-omni ikholopov-omni force-pushed the bazel7js branch 5 times, most recently from 379cf44 to 665c009 Compare June 17, 2026 13:53
@ikholopov-omni ikholopov-omni marked this pull request as ready for review June 17, 2026 14:32
@ikholopov-omni ikholopov-omni requested a review from a team as a code owner June 17, 2026 14:32
rafal-hawrylak
rafal-hawrylak requested changes Jun 17, 2026
View reviewed changes
Comment thread packages/rollup.config.js Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rafal-hawrylak rafal-hawrylak rafal-hawrylak approved these changes

@kolina kolina Awaiting requested review from kolina

Assignees

@ikholopov-omni ikholopov-omni

Labels

Internal Cleanup

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ikholopov-omni @rafal-hawrylak