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

fix(ci): fix e2e test failures after merge #125

Merged

Conversation

welkin22
Copy link
Contributor

Description

After merging upstream 1.3.0, there were significant changes in the e2e test, and some cases failed to pass the test and need to be fixed.

Rationale

fix e2e test case

Example

none

Changes

Notable changes:

  • Fix the e2e case not passing issue caused by modifying Devnet.
  • ...

@github-actions github-actions bot requested review from krish-nr and redhdx February 23, 2024 02:13
@owen-reorg
Copy link
Collaborator

The ci is not triggered in this PR. Can we enable it to check the effect before it's merged (by add adding trigger condition of this branch or manually trigger)?

@welkin22
Copy link
Contributor Author

The ci is not triggered in this PR. Can we enable it to check the effect before it's merged (by add adding trigger condition of this branch or manually trigger)?

I tested in my repository and currently encountered some problems that are still being solved.

@welkin22 welkin22 requested a review from owen-reorg March 8, 2024 02:11
@bnoieh bnoieh merged commit 0d298b1 into bnb-chain:merge-upstream-v1.3.0 Mar 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants