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

[Examples/Move][EASY] Change Directory #14429

Merged
merged 1 commit into from
Oct 25, 2023
Merged

[Examples/Move][EASY] Change Directory #14429

merged 1 commit into from
Oct 25, 2023

Conversation

amnn
Copy link
Member

@amnn amnn commented Oct 25, 2023

Description

As in title

Test Plan

👀

@vercel
Copy link

vercel bot commented Oct 25, 2023

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

Name Status Preview Updated (UTC)
mysten-ui ✅ Ready (Inspect) Visit Preview Oct 25, 2023 6:39pm
sui-typescript-docs ✅ Ready (Inspect) Visit Preview Oct 25, 2023 6:39pm
3 Ignored Deployments
Name Status Preview Updated (UTC)
explorer ⬜️ Ignored (Inspect) Oct 25, 2023 6:39pm
multisig-toolkit ⬜️ Ignored (Inspect) Oct 25, 2023 6:39pm
sui-kiosk ⬜️ Ignored (Inspect) Oct 25, 2023 6:39pm

@amnn
Copy link
Member Author

amnn commented Oct 25, 2023

As per discussion last week -- changing the name of the directory where Move examples are held.

  • @ronny-mysten, could you do a pass over the docs to check if there are links to the old location?
  • @healthydeve, this change is going to conflict with your PR modernising one of the examples, could you make the corresponding rename on your side as well?

@amnn amnn merged commit 55b794e into main Oct 25, 2023
35 checks passed
@amnn amnn deleted the amnn/move-examples branch October 25, 2023 22:23
@oxade oxade mentioned this pull request Oct 26, 2023
7 tasks
oxade added a commit that referenced this pull request Oct 26, 2023
## Description 

Fixes breakage due to #14429

## Test Plan 

Existing
---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
amnn added a commit that referenced this pull request Oct 27, 2023
Move tests are run as part of the Rust tests for the Sui Framework,
but they should also be run when there are only Move changes.

Surfaced because a recent PR (#14429) Moved some Move tests causing CI
to fail on `main`, and this wasn't picked up because diff tests hadn't
been run.

Test Plan:

CI
amnn added a commit that referenced this pull request Oct 27, 2023
Move tests are run as part of the Rust tests for the Sui Framework,
but they should also be run when there are only Move changes.

Surfaced because a recent PR (#14429) Moved some Move tests causing CI
to fail on `main`, and this wasn't picked up because diff tests hadn't
been run.

Test Plan:

CI
amnn added a commit that referenced this pull request Oct 27, 2023
## Description

Move tests are run as part of the Rust tests for the Sui Framework, but
they should also be run when there are only Move changes.

Surfaced because a recent PR (#14429) Moved some Move tests causing CI
to fail on `main`, and this wasn't picked up because diff tests hadn't
been run.

## Test Plan

CI
jonas-lj pushed a commit to jonas-lj/sui that referenced this pull request Nov 2, 2023
## Description 

As in title

## Test Plan 

👀
jonas-lj pushed a commit to jonas-lj/sui that referenced this pull request Nov 2, 2023
## Description 

Fixes breakage due to MystenLabs#14429

## Test Plan 

Existing
---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
jonas-lj pushed a commit to jonas-lj/sui that referenced this pull request Nov 2, 2023
## Description

Move tests are run as part of the Rust tests for the Sui Framework, but
they should also be run when there are only Move changes.

Surfaced because a recent PR (MystenLabs#14429) Moved some Move tests causing CI
to fail on `main`, and this wasn't picked up because diff tests hadn't
been run.

## Test Plan

CI
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.

4 participants