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

fixing continuous integration #218

Merged
merged 3 commits into from
Apr 10, 2024
Merged

fixing continuous integration #218

merged 3 commits into from
Apr 10, 2024

Conversation

jac18281828
Copy link
Contributor

closes #217

  1. fixing solidity formatting, using solidity standard conventions, double quotes, 4 spaces, etc
  2. fixes failing test
  3. prettier is checked in ci

@jac18281828 jac18281828 self-assigned this Apr 5, 2024
@jac18281828 jac18281828 force-pushed the jac/cibuildfix branch 2 times, most recently from e4889cc to 891f3f6 Compare April 5, 2024 21:54
@jac18281828 jac18281828 marked this pull request as draft April 5, 2024 21:54
Fixes for NodeELRewardVault.t

Fixes for solidity formatting:
 - bracketSpacing
 - double quote
 - no tabs

Fixes test_withdraw in NodeELRewardVault.t.sol

Pass operator address as argument to NodeRegistryMock

rebase changes from mainnet_V0

fixing hoax prank issues

ValidatorWithdrawalVaultTest is passing

PermissionlessNodeRegistryTest.test_markReadyToDepositValidator is passing

PermissionlessNodeRegistryTest.test_getAllActiveValidatorsWithZeroPageNumber is passing

PermissionlessNodeRegistryTest.test_getAllActiveValidators is passing

PermissionlessPoolTest.test_setCommisionFeesWithInvalidInput is passing

PermissionedPoolTest.test_StakeUserETHToBeaconChain is passing

OperatorRewardsCollectorTest passing

All tests passing
@jac18281828 jac18281828 force-pushed the jac/cibuildfix branch 3 times, most recently from 16cd620 to 5caad1d Compare April 8, 2024 17:55
@jac18281828 jac18281828 marked this pull request as ready for review April 8, 2024 17:58
@jac18281828
Copy link
Contributor Author

Tests are now passing!

  • lint is running but failing. I will address this soon/next.
  • code coverage is added but the app needs configuration for Stader-labs org
  • cleaned up all the GitHub actions

- adding support for code coverage analytics
- SDUtilityPoolTest.test_RequestWithdraw - cleanup test with fuzzer range issues
- adding manual workflow dispatch
@jac18281828 jac18281828 removed the request for review from manoj9april April 9, 2024 13:43
Copy link

@galacticminter galacticminter left a comment

Choose a reason for hiding this comment

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

LGTM

@jac18281828 jac18281828 merged commit f545abe into mainnet_V0 Apr 10, 2024
1 of 5 checks passed
@jac18281828 jac18281828 deleted the jac/cibuildfix branch April 10, 2024 19:14
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.

Continuous Integration Tests Failing
2 participants