Skip to content

move cli logic -> cli-core so it can be used as a node package#598

Open
elee1766 wants to merge 6 commits intowormhole-foundation:mainfrom
elee1766:cli-updates
Open

move cli logic -> cli-core so it can be used as a node package#598
elee1766 wants to merge 6 commits intowormhole-foundation:mainfrom
elee1766:cli-updates

Conversation

@elee1766
Copy link
Contributor

No description provided.

@elee1766 elee1766 marked this pull request as ready for review March 17, 2025 20:10
@nik-suri nik-suri requested review from evgeniko and kcsongor March 18, 2025 16:45
@kcsongor
Copy link
Contributor

kcsongor commented Apr 4, 2025

When running ./cli/install.sh locally on this branch, I get

bun link v1.1.20 (ae194892)
error: GET https://registry.npmjs.org/@wormhole-foundation%2fntt-cli-core - 404
error: @wormhole-foundation/ntt-cli-core@* failed to resolve

@elee1766
Copy link
Contributor Author

elee1766 commented Apr 4, 2025

When running ./cli/install.sh locally on this branch, I get

bun link v1.1.20 (ae194892)
error: GET https://registry.npmjs.org/@wormhole-foundation%2fntt-cli-core - 404
error: @wormhole-foundation/ntt-cli-core@* failed to resolve

correct. install cannot work because the bun can only fetch from npm from what i understand, which i cannot publish to.

@kcsongor
Copy link
Contributor

kcsongor commented Apr 4, 2025

hm, isn't there a way to set the workspace up in the top level directory so that it gets installed locally?

@elee1766
Copy link
Contributor Author

elee1766 commented Apr 4, 2025

i don't know because i dont think the top level repo is bun. bun seems to only be used for the cli?

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