-
Notifications
You must be signed in to change notification settings - Fork 77
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
Enhancement: Moved from react to Vite #64
Conversation
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 🚨 Potential security issues found in this pull request. To accept the risk, merge this PR and you will not be notified again. Bot CommandsTo ignore an alert, reply with a comment starting with
📜 Install scriptsInstall scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts. Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.
Pull request alert summary
📊 Modified Dependency Overview:
🚮 Removed packages: [email protected] |
Your yarn.lock checksum here is wrong: Please use
in your yarn.lock instead to pass tests @Ocolus1 |
There's still errors: |
@Divide-By-0 react-scripts was removed intentionally, the test will not pass. It was necessary to convert to Vite. |
Try these edits in package.json and test to ensure the scripts work for instance: @Ocolus1
For your snarkjs fork, you need to fork sampritipanda's fork, not the original. |
Closed by #76. |
No description provided.