Skip to content

Allowed end-developers to install @embroider/try#8

Closed
ijlee2 wants to merge 1 commit intoembroider-build:mainfrom
ijlee2:feature-allow-installation
Closed

Allowed end-developers to install @embroider/try#8
ijlee2 wants to merge 1 commit intoembroider-build:mainfrom
ijlee2:feature-allow-installation

Conversation

@ijlee2
Copy link
Copy Markdown

@ijlee2 ijlee2 commented Feb 12, 2026

Background

The README for v1.0.1 claims that one can run pnpm @embroider/try, but this isn't possible (to my knowledge) due to the package name starting with @.

By providing an alias, we can provide end-developers the option to install @embroider/try in a package as a development dependency. (That is, they can fix the version and don't have to use npx, pnpx, etc.)

pnpm embroider-try <options>

@ijlee2 ijlee2 marked this pull request as ready for review February 12, 2026 08:05
@NullVoxPopuli
Copy link
Copy Markdown

I think pnpm -s @embroider/try list may have been a typo

if installed, pnpm try list works

@ijlee2
Copy link
Copy Markdown
Author

ijlee2 commented Mar 5, 2026

if installed, pnpm try list works

Yep, I checked this in ijlee2/ember-container-query@2163b14. I had misunderstood how @embroider/try is meant to be used in CI, because the first command in README showed pnpm, not pnpx. I'll try to fix this in #10.

@ijlee2 ijlee2 closed this Mar 5, 2026
@ijlee2 ijlee2 deleted the feature-allow-installation branch March 5, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants