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(move-analyzer): fix crash if open a standalone Move file #4791

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

valeriyr
Copy link
Contributor

Description of change

Since the upstream pulling is postponed it is better to push this simple change to fix the crash and update the extension.

This change is a part of the following upstream PR which is mentioned in the issue:
MystenLabs/sui#20510

Links to any relevant issues

fixes #4387

@valeriyr valeriyr added sc-platform Issues related to the Smart Contract Platform group. vm-language Issues related to the VM & Language Team labels Jan 15, 2025
@valeriyr valeriyr requested a review from miker83z January 15, 2025 10:59
@valeriyr valeriyr self-assigned this Jan 15, 2025
@valeriyr valeriyr requested a review from a team as a code owner January 15, 2025 10:59
Copy link

vercel bot commented Jan 15, 2025

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

Name Status Preview Comments Updated (UTC)
apps-backend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 10:59am
apps-ui-kit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 10:59am
rebased-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 10:59am
wallet-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 10:59am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sc-platform Issues related to the Smart Contract Platform group. vm-language Issues related to the VM & Language Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[move-ide] The move analyzer crashes if the manifest file is missing
4 participants