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.

Sign up for GitHub

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

Jump to bottom

Fix: bytecode verification [review suggestions] #763

Merged

bulbozaur merged 2 commits into feature/bytecode-verificator-cli from fix/review-proposals

May 1, 2023

Contributor

TheDZhon commented May 1, 2023

Forbid run as root
Require nc to be installed
Avoid unset variables
Replace sig numbers with aliases
cd using $PWD
multiline ganache launcher


          fix: review suggestions

d571a44

TheDZhon requested a review from bulbozaur

May 1, 2023 17:46

TheDZhon mentioned this pull request

Feature/bytecode verificator cli #761

Merged


          fix: use sed

6d5018e

bulbozaur approved these changes

View reviewed changes

bulbozaur merged commit c785b27 into feature/bytecode-verificator-cli

TheDZhon deleted the fix/review-proposals branch

May 1, 2023 18:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet