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

fix: update Sui environment setup #18303

Merged
merged 1 commit into from
Dec 22, 2024
Merged

Conversation

emmanuel-ferdman
Copy link
Contributor

Description

PR #17253 moved the location of the Sui environment setup file. This PR adjusts sources to changes.

Test plan

Checked the doc.


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):
  • Indexer:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:

Copy link

vercel bot commented Jun 18, 2024

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 Dec 22, 2024 11:01am
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Dec 22, 2024 11:01am
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Dec 22, 2024 11:01am
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Dec 22, 2024 11:01am

Copy link

vercel bot commented Jun 18, 2024

@emmanuel-ferdman is attempting to deploy a commit to the Mysten Labs Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 18, 2024
@ronny-mysten
Copy link
Contributor

Thanks @emmanuel-ferdman !

@ronny-mysten ronny-mysten enabled auto-merge (squash) August 19, 2024 15:44
@github-actions github-actions bot removed the Stale label Aug 20, 2024
@emmanuel-ferdman
Copy link
Contributor Author

Hi @ronny-mysten @aschran @Jordan-Mysten @williampsmith @amnn, can you please review this small PR when you get the chance?

@amnn amnn disabled auto-merge December 18, 2024 15:56
@amnn
Copy link
Member

amnn commented Dec 18, 2024

Looks like because the PR is so old, we'll need to rebase it and re-submit so CI can run. @emmanuel-ferdman, could you do that? Then I'll be able to make sure it gets landed.

Signed-off-by: Emmanuel Ferdman <[email protected]>
@emmanuel-ferdman
Copy link
Contributor Author

@amnn should be good now - the branch is updated to latest changes in main. Thank you for the review.

@emmanuel-ferdman emmanuel-ferdman temporarily deployed to sui-typescript-aws-kms-test-env December 22, 2024 12:13 — with GitHub Actions Inactive
@amnn amnn merged commit 01e3e6a into MystenLabs:main Dec 22, 2024
51 checks passed
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.

3 participants