Skip to content

Fix bench post-install #1732

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
merged 7 commits into from
Mar 5, 2025
Merged

Fix bench post-install #1732

merged 7 commits into from
Mar 5, 2025

Conversation

NullVoxPopuli
Copy link
Contributor

No description provided.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 4, 2025 •
edited
Loading

This PRmain
Dev 
583K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 19K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner

583K └─┬ .
169K   ├── runtime
160K   ├── syntax
100K   ├── compiler
 58K   ├── opcode-compiler
 27K   ├── manager
 19K   ├── validator
 11K   ├── program
8.9K   ├── reference
7.2K   ├── destroyable
6.3K   ├── util
4.3K   ├── node
3.4K   ├── global-context
2.5K   ├── wire-format
1.0K   ├── vm
969B   ├── encoder
844B   ├── vm-babel-plugins
606B   └── owner
Prod 
230K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
4.8K   ├── program
4.0K   ├── validator
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner

230K └─┬ .
 70K   ├── syntax
 63K   ├── runtime
 48K   ├── compiler
 18K   ├── opcode-compiler
7.9K   ├── manager
4.8K   ├── program
4.0K   ├── validator
3.6K   ├── reference
2.4K   ├── util
2.1K   ├── node
1.6K   ├── wire-format
1.5K   ├── destroyable
737B   ├── vm
594B   ├── global-context
516B   ├── encoder
469B   ├── vm-babel-plugins
155B   └── owner

@NullVoxPopuli NullVoxPopuli merged commit 5474c62 into main Mar 5, 2025
9 checks passed
@NullVoxPopuli NullVoxPopuli deleted the fix-post-install branch March 5, 2025 16:20
@github-actions github-actions bot mentioned this pull request Mar 5, 2025
Merged
git-online-helper bot added a commit to patricklx/glimmer-vm that referenced this pull request May 13, 2025
@git-online-helper
Move prettier tests to the smoke-tests project
3582aef 
Update lockfile

x

ope

Merge pull request glimmerjs#1726 from glimmerjs/put-prettier-testing-in-the-smoke-tests-project

Move prettier tests to the smoke-tests project

Update release-plan

Don't forget repo meta update

Printer bug: empty string literal args are dropped

proposed fix

lint:fix

Merge pull request glimmerjs#1727 from glimmerjs/update-release-plan

Update release-plan

Merge pull request glimmerjs#1722 from glimmerjs/printer-empty-string-literal

Printer bug: empty string literal args are dropped

Set node-version to 22

Merge pull request glimmerjs#1728 from glimmerjs/set-node-version-to-22

Set node-version to 22 in plan-release/publish

Prepare Release  using 'release-plan'

Merge pull request glimmerjs#1715 from glimmerjs/release-preview

Prepare Release

Revert "Prepare Release"

Merge pull request glimmerjs#1729 from glimmerjs/revert-1715-release-preview

Revert "Prepare Release"

Go back to using configs from before glimmerjs#1727

Merge pull request glimmerjs#1730 from glimmerjs/fix-release-plan

fix release-plan

Prepare Release  using 'release-plan'

Merge pull request glimmerjs#1731 from glimmerjs/release-preview

Prepare Release

Fix bench post-install

ignore-workspace -> ignore-workspace-root-check

Force CI=false for bench

Update bench-packages.mts

Use postinstall script to opt out of postinstall

+x

bah

Merge pull request glimmerjs#1732 from glimmerjs/fix-post-install

Fix bench post-install

Restore {{debugger}} behavior

Merge pull request glimmerjs#1734 from glimmerjs/restore-debugger

Restore {{debugger}} behavior

Prepare Release  using 'release-plan'

Manually update the release-preview to force @glimmer/runtime to release

Add pkgJSONPath

Add constraints array (empty)

Add entry to constraints arary

Add impact

Merge pull request glimmerjs#1733 from glimmerjs/release-preview

Prepare Release

Force release

Merge pull request glimmerjs#1735 from glimmerjs/force-release

Force release

pnpm repo:update:metadata

Merge pull request glimmerjs#1736 from glimmerjs/nvp/update-meta

pnpm repo:update:metadata

Removing editor.rulers

This drives me batty every time I open this repo. I think it's fair to say it falls into opinionated IDE settings that can stay in a personal config instead of the repo.

Merge pull request glimmerjs#1738 from glimmerjs/editor-rulers

Removing editor.rulers

Re-add package.json#types for tsconfig's moduleResolution=node10

Merge pull request glimmerjs#1741 from glimmerjs/add-back-types-for-moduleResolution=node10

Re-add support for tsconfig's moduleResolution=node10

Prepare Release  using 'release-plan'

Merge pull request glimmerjs#1737 from glimmerjs/release-preview

Prepare Release

Upgrade handlebars parser

Upgrade to nppm 10.6.5

Merge pull request glimmerjs#1743 from glimmerjs/upgrade-handlebars-parser

Upgrade handlebars parser

Merge pull request glimmerjs#1744 from glimmerjs/nvp/update-pnpm

Upgrade to pnpm 10.6.5

test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
internal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@NullVoxPopuli