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.

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

hardhat e2e for dsm v3 #266

Open
wants to merge 61 commits into
base: develop
Choose a base branch
from
Open

hardhat e2e for dsm v3 #266

wants to merge 61 commits into from

Conversation

Amuhar
Copy link
Contributor

@Amuhar Amuhar commented Oct 10, 2024 •
edited
Loading

No description provided.

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 10, 2024
View reviewed changes
test/helpers/pg.ts Fixed Show fixed Hide fixed
test/helpers/pg.ts Fixed Show fixed Hide fixed
@Amuhar Amuhar marked this pull request as draft October 10, 2024 13:20
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 31, 2024
View reviewed changes
test/helpers/pg.ts Fixed Show fixed Hide fixed
test/helpers/pg.ts Fixed Show fixed Hide fixed
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 5, 2024
View reviewed changes
test/helpers/pg.ts Dismissed Show dismissed Hide dismissed
test/helpers/pg.ts Dismissed Show dismissed Hide dismissed
@Amuhar Amuhar marked this pull request as ready for review November 6, 2024 17:49
@Amuhar Amuhar requested a review from a team as a code owner November 7, 2024 10:54
Amuhar and others added 28 commits November 8, 2024 10:38
@Amuhar
fix: add err
dd7ed7c
@Amuhar
fix: kapi
c06274e
@Amuhar
fix: enable other duplicates tests
115ac94
@Amuhar
fix: enable invalid-keys-v3 tests
002e9ea
@Amuhar
fix: add logs to hardhat
184599e
@Amuhar
fix: hardhat kill
d9324c1
@Amuhar
fix: check port
91c05e0
@Amuhar
fix: check port
0777e7b
@Amuhar
fix: check port
573c253
@Amuhar
fix: enable front-run tests
7b9dc8b
@Amuhar
fix: ebable front-run tests
0ac5029
@Amuhar
fix: read envs from config
07b51ee
@Amuhar
fix: read envs from config
5d4a537
@eddort
feat: separated ci jobs
71f0ab7
@eddort
fix: chain config
95253ba
@eddort
feat: run tests in parallel
1d3d9ba
@Amuhar
Merge pull request #273 from lidofinance/feat/val-1135-e2e-impr-ci-au…
dd083d6 
…tomation

feat: separated ci jobs
@Amuhar
fix: enable tests
1703eaf
@Amuhar
fix: local test run on macos
6ae3d29
@Amuhar
removes unnecessary code
e7be612
@Amuhar
fix: data bus tests on hardhat
3324225
@Amuhar
fix: unit tests workflow
ff4c269
@Amuhar
fix: add locator env for unit tests workflow
c0e39ec
@Amuhar
fix: temp disable e2e
17d72f7
@Amuhar
fix: enable e2e workflow
3d115ed
@Amuhar
fix: connect to hardhat
7efcc63
@Amuhar
fix: removed unnecessary comments
a768ee3
@Amuhar
fix: port name
3d5c79f
@eddort
Copy link
Member

eddort commented Nov 27, 2024

Great job, thank you very much! ❤️

rnmsslido
rnmsslido approved these changes Nov 27, 2024
View reviewed changes
Copy link

@rnmsslido rnmsslido left a comment

Choose a reason for hiding this comment

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

LGTM for changes on .github/workflows.
Before merging, please ensure additional reviews are obtained for the other changes covered in this PR.

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

@rnmsslido rnmsslido rnmsslido approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Amuhar @eddort @rnmsslido