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

Able to run multiple commands #2799

Merged
merged 8 commits into from
Feb 1, 2024

Conversation

aoikurokawa
Copy link
Contributor

close #2771

Copy link

vercel bot commented Jan 29, 2024

@aoikurokawa is attempting to deploy a commit to the coral-xyz Team on Vercel.

A member of the Team first needs to authorize it.

@acheroncrypto acheroncrypto added enhancement New feature or request cli labels Jan 30, 2024
Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this works for ; syntax but it also feels like we're reimplementing bash syntax here e.g. what happens if people use && or set env variables?

Is it possible to run the input like a user would in their terminal so that we can avoid implementing custom logic based on input?

@aoikurokawa
Copy link
Contributor Author

Thank you for the review.
I misunderstood the issue initially. I thought that anchor run script could not execute multiple commands, but in reality, it was the anchor test command that had this limitation.
Therefore, I modified the code within the test command.

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@acheroncrypto acheroncrypto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@acheroncrypto acheroncrypto merged commit 94fde24 into coral-xyz:master Feb 1, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to run multiple commands with anchor test.
2 participants