Skip to content
This repository has been archived by the owner on Jun 15, 2024. It is now read-only.

Getting started tweaks #70

Open
wants to merge 4 commits into
base: mhofman/hack-getting-started
Choose a base branch
from

Conversation

michaelfig
Copy link
Member

No description provided.

@michaelfig michaelfig self-assigned this Nov 16, 2023
Copy link
Member

@mhofman mhofman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either we need to redo the resolutions, or accept the fact we'll need to rely on the babel patches for now

yarn.lock Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh no we have a bunch of @endo and @babel deps that got bumped up!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I'd prefer to rely on resolutions in addition to the yarn.lock.

README.md Outdated
cd $HOME
# Install the dapp into a directory named demo
npm create @agoric/dapp@getting-started-dev init --dapp-template dapp-card-store --dapp-branch mhofman/hack-getting-started demo
npm create @agoric/dapp -- --dapp-template dapp-card-store --dapp-branch mhofman/hack-getting-started demo
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove the @getting-started-dev tag? until we publish a new latest, don't we need it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I published @agoric/create-dapp already since it uses the agoric@getting-started already, and only relies on agoric init to do the right thing, which it does.

Copy link
Member

@mhofman mhofman Nov 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The latest tag is technically older: https://www.npmjs.com/package/@agoric/create-dapp?activeTab=versions

If we end up making changes, they wouldn't be picked up

@mhofman
Copy link
Member

mhofman commented Dec 29, 2023

FYI @dckc some dapp work for NPM compatibility that Michael and I did during the hackathon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants