Skip to content
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

[suiop][env] fix messaging, remove error #21354

Merged
merged 2 commits into from
Feb 25, 2025
Merged

Conversation

after-ephemera
Copy link
Contributor

Description

  • remove old error checking for an env file
  • add a new message to indicate that a subshell is being created

Test plan

suiop-cli [jkj/suiop-env-msg-fix] suiop env                    
> Select an environment: mysten/default/vultr-api
Starting a subshell with the environment set... exit the shell (usually ctrl-d) to exit the env
error This project's package.json defines "packageManager": "yarn@[email protected]". However the current global version of Yarn is 1.22.22.                                                                                                                                                                                

Presence of the "packageManager" field indicates that the project is meant to be used with Corepack, a tool included by default with all official Node.js distributions starting from 16.9 and 14.19.
Corepack must currently be enabled by running corepack enable in your terminal. For more information, check out https://yarnpkg.com/corepack.
done initializing 🎉
suiop-cli [jkj/suiop-env-msg-fix] 

Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • gRPC:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:

@after-ephemera after-ephemera requested a review from a team February 25, 2025 21:59
Copy link

vercel bot commented Feb 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 25, 2025 11:08pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 11:08pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Feb 25, 2025 11:08pm

@after-ephemera after-ephemera temporarily deployed to sui-typescript-aws-kms-test-env February 25, 2025 23:06 — with GitHub Actions Inactive
@after-ephemera after-ephemera merged commit a5a71ef into main Feb 25, 2025
47 checks passed
@after-ephemera after-ephemera deleted the jkj/suiop-env-msg-fix branch February 25, 2025 23:31
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