Skip to content

Working toward glimmer-vm repo integration #20908

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ef4 merged 5,264 commits into from
Jan 6, 2026
Merged

Working toward glimmer-vm repo integration #20908

ef4 merged 5,264 commits into from
Jan 6, 2026

Conversation

@ef4
Copy link
Contributor

@ef4 ef4 commented May 6, 2025

This is a work-in-progress on harmonizing the glimmer-vm repo with this repo so they can eventually merge.

@NullVoxPopuli
Merge pull request #1696 from glimmerjs/update-meta
c059614 
Update package meta post-publish and tell the auto-publish infra to do this for us
@NullVoxPopuli
Fix exports
2cd9dce
@NullVoxPopuli
Merge pull request #1698 from glimmerjs/fix-exports
5b2ebc2 
Fix exports
@NullVoxPopuli @github-actions
Prepare Release using 'release-plan'
d098303
@NullVoxPopuli
Merge pull request #1697 from glimmerjs/release-preview
72ce402 
Prepare Release
@NullVoxPopuli
Re-enable CJS for @glimmer/compiler (used by babel plugin)
a4b9110
@NullVoxPopuli
cleanup
99a7f03
@NullVoxPopuli
No types for cjs
dceee72
@NullVoxPopuli
lint:fix
4eb7566
@NullVoxPopuli
Key off require instead of cjs
5c0fae1
@NullVoxPopuli
Re-add cjs require
8255e9b
@NullVoxPopuli
Update meta to support cjs
3bb89eb
@NullVoxPopuli
Merge pull request #1699 from glimmerjs/cjs-womp
52e4509 
Re-enable CJS for @glimmer/compiler (used by babel plugin)
@NullVoxPopuli @github-actions
Prepare Release using 'release-plan'
2971acf
@NullVoxPopuli
Merge pull request #1700 from glimmerjs/release-preview
a306d62 
Prepare Release
@NullVoxPopuli
Rollup the types
a37f8de
@NullVoxPopuli
Don't build types for the vm-babel-plugins
ed9d6dd
@NullVoxPopuli
Update metadata
6d67aa5
@NullVoxPopuli
Remove the 'ms' package as it is no longer used
42c3c39
@NullVoxPopuli
Remove types from vm-babel-plugins for now
7a5e6c7
@NullVoxPopuli
Merge pull request #1701 from glimmerjs/rollup-types
b295fc6 
Rollup the types (solves missing types from unpublished packages)
@NullVoxPopuli
Add README to @glimmer/interfaces to try to force a release of depend…
83a0f0b 
…ent packages
@NullVoxPopuli
Merge pull request #1703 from glimmerjs/each-built-package-must-depen…
61c1d7c 
…d-on-build-tools

Add README to @glimmer/interfaces to try to force a release of dependent packages
@NullVoxPopuli @github-actions
Prepare Release using 'release-plan'
8e7ece0
@NullVoxPopuli
Merge pull request #1702 from glimmerjs/release-preview
cb9c617 
Prepare Release
@NullVoxPopuli
Update build config and meta
1367721
@NullVoxPopuli
Update plan-release
bc31a41
@NullVoxPopuli
Merge pull request #1704 from glimmerjs/fix-types-rollup
ad7f15f 
Fix types rollup
@NullVoxPopuli @github-actions
Prepare Release using 'release-plan'
0ec0b39
@NullVoxPopuli
Merge pull request #1705 from glimmerjs/release-preview
591c61f 
Prepare Release
@ef4
Copy link
Contributor Author

ef4 commented Dec 9, 2025 •
edited
Loading

This now has merged the history and unified the test suites.

  • test logging is missing on CI (not specific to this branch)
  • there are four remaining failures (that we can't see in detail) on the production mode tests
  • lints are skipped, we want to move the glimmer-vm packages into the real packages dir first in order to unify their type-build tsconfigs.

ef4 and others added 20 commits December 16, 2025 14:04
@ef4
use real tracking reset in tests, not just debug tracking reset (whic…
4871b5e 
…h doesn't work in prod builds)
@ef4
guarding cross-browser behavior tests
3012cd7
@ef4
unused now
5791661
@ef4
cleaning up link scripts
53a30ae
@ef4
moving glimmer-vm packages into shared packages subdir
720206c
@ef4
cleaning up unused infrastructure and type errors
9406100
@ef4
ts 5.9 doesn't behave the same here
356afef
@ef4
type publishing integration
c108b7b
@ef4
adjust postprocessing filter
f5640a3
@ef4
more precise exclude for glimmer tests
8cf5f8e
@ef4
Merge remote-tracking branch 'origin/main' into glimmervm-submodule
1d8c7eb
@ef4
drop vm-babel-plugins
bf3bc3c
@ef4
keep glimmer packages in "exposedDependencies" and not also "packages"
9774499
@NullVoxPopuli
Update ESLint
91fc538
@NullVoxPopuli
run lint:fix -- only 6 problems left
f26ae09
@NullVoxPopuli
Fix the 6 issues. run prettier.
8bf8cff
@NullVoxPopuli
Remove the v1 handlebars AST from YUIDoc, so that the lint:docs task …
c8e4bd4 
…passes
@NullVoxPopuli
Fix
d1a772d
@ef4
Merge pull request #21020 from NullVoxPopuli/nvp/glimmervm-merge/esli…
b0e49df 
…nt-updates

Re-activate linting
@ef4
build glimmer/env for backward compat
ba01774 
implicitModules uses it in classic builds when AMD is disabled
@ef4 ef4 marked this pull request as ready for review January 6, 2026 16:04
@ef4 ef4 merged commit 4abca82 into main Jan 6, 2026
29 checks passed
@ef4 ef4 deleted the glimmervm-submodule branch January 6, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kategengler kategengler kategengler approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@ef4 @kategengler @NullVoxPopuli @chancancode @wycats @boris-petrov