-
Notifications
You must be signed in to change notification settings - Fork 9
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
E2e test for sign-multichain #244
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
2f3c270
to
4ec827b
Compare
4ec827b
to
c1f5cb8
Compare
a69544f
to
8ee1a5f
Compare
369682c
to
643839d
Compare
Send secrets to process.env Send .env to playwright config
9e33033
to
6c9d844
Compare
.github/workflows/playwright.yml
Outdated
run: | | ||
echo "ETH_PROVIDER_URL_TESTNET=${{ secrets.ETH_PROVIDER_URL_TESTNET }}" >> $GITHUB_ENV | ||
echo "BNB_PROVIDER_URL_TESTNET=${{ secrets.BNB_PROVIDER_URL_TESTNET }}" >> $GITHUB_ENV | ||
echo "BTC_PROVIDER_URL_TESTNET=${{ secrets.BTC_PROVIDER_URL_TESTNET }}" >> $GITHUB_ENV |
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.
why does it have to be passed down to .env instead of directly feed into process.env like
MULTICHAIN_TEST_ACCOUNT_ID: ${{ secrets.MULTICHAIN_TEST_ACCOUNT_ID }}
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.
@hcho112 that's right.
Fixed.
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.
Left one comment for clarification but rest looks good
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.
LGTM
This is the PR for sign-multichain e2e tests