-
Notifications
You must be signed in to change notification settings - Fork 52
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
Added Tezos Infra Module #922
Conversation
@@ -0,0 +1,7 @@ | |||
const CMDS = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@namesty I'm confused by how these js modules get used, as I don't see them referenced in the docker-compose.yaml or Dockerfile.
Are they useful to users in some way? If so, how can they access them? Is this something common you think we should do (bundle helper scripts w/ the infra modules)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, they do seem unrelated, I'll remove them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do tezos have test-env-js like we have for default test-env?
This will be out of scope for this PR but would like to suggest that we should have better way to check whether all the services are started or stopped instead of sleeping for arbitrary time and hope it will be started
Migrate to Tezos Jakarta protocol
Going to close this inactivity. Additionally we should publish this within another repo instead of adding it as a baked-in default within the CLI. |
Closes polywrap/wrap-integrations#52
This PR adds a Tezos blockchain environment, as a default infra module