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

Compliance tests verification #103

Merged
merged 39 commits into from
Sep 12, 2024
Merged

Conversation

pablodeymo
Copy link
Collaborator

No description provided.

pablodeymo and others added 15 commits September 11, 2024 16:54
- Added test-coverage to CI, excluding `fireblocks` tests
- Added build step to CI
- Removed foundry installation from CI
- Fixed flaky test in `operatorsinfo_inmemory`
- Added clippy warning for `unwrap_used`, `expect_used`, `todo`
- Fixed typos, added error handling and remove unused code

---------

Co-authored-by: Pablo Deymonnaz <[email protected]>
- Add foundry installation to CI
- Remove CI environments to avoid the github deploy messages
- Updated coverage section of the README.
- Updated coverage target in Makefile to generate html report
Small PR refactoring get_transaction_status function and fixing an
unused import.
@TomasArrachea TomasArrachea marked this pull request as ready for review September 12, 2024 18:00
@pablodeymo pablodeymo changed the title [WIP] Compliance tests verification Compliance tests verification Sep 12, 2024
@TomasArrachea TomasArrachea merged commit 982758b into Layr-Labs:main Sep 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants