-
Notifications
You must be signed in to change notification settings - Fork 215
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.
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
no supported workflow for installing from npm #3857
Comments
We can certainly make Be aware that if you want to use an existing checkout of a. Use Pick your poison. |
The usual workflow for this is
|
@mhofman reports that we're not quite there yet: #4129 (comment) |
I ran into another person trying to do |
We disabled the CI test that uses |
I'm not surprised either; https://docs.agoric.com/guides/getting-started/ is the only currently supported path. This open issue is just an acknowledgement that we're behind market norms in this respect. |
I tried using the SDK as normal npm packages in the context of some recent work on a smart-wallet style app starter, but I hit:
looks like So I went back to p.s. my WIP: dc-test-contract agoric-labs/dapp-join-game@1a653c0 |
We now have a supported workflow based on https://docs.agoric.com/guides/getting-started/ documents it as of 1cf164c |
Describe the bug
We don't have a supported workflow of building dapps by installing packages such as
@agoric/zoe
from npm. Only building from source is adequately supported.For example, the agoric cli is installed using
yarn link-cli
, unlike other npm tools that are installed usingnpm install -g
or the like.To Reproduce
If you run
agoric install
from inside anagoric-sdk
, you're in a world of hurt.If you then make a new
agoric-sdk
clone, forgetting toyarn link-cli
leads to difficult-to-diagnose version skew (this cost @dtribble a bunch of time today).npx agoric init demo
doesn't work:running
agoric
with a pre-ESM version gives a pretty bad diagnostic:Expected behavior
like other dev tools in the npm world
cc @mhofman @katelynsills @warner @michaelfig
The text was updated successfully, but these errors were encountered: