Skip to content
This repository was archived by the owner on Feb 7, 2025. It is now read-only.

Feature/permit lock and process expired locks tests#42

Open
0xhafa wants to merge 11 commits intomainfrom
feature/permit-lock-and-process-expired-locks-tests
Open

Feature/permit lock and process expired locks tests#42
0xhafa wants to merge 11 commits intomainfrom
feature/permit-lock-and-process-expired-locks-tests

Conversation

@0xhafa
Copy link
Copy Markdown
Collaborator

@0xhafa 0xhafa commented Feb 23, 2023

PR Review:

  • Add Permit functionality to ClaimAndLock() in Relocker
  • Remove processExpiredLocksAndRelockRewards() from Relocker (not possible with the current implementation of RLBTRFLY)
  • Add test cases for ClaimAndLock() with permit

@drahrealm
Copy link
Copy Markdown
Contributor

Can you try to check and fix the failed test in CircleCI? 🙏

Comment thread contracts/core/Relocker.sol Outdated
@0xhafa 0xhafa force-pushed the feature/permit-lock-and-process-expired-locks-tests branch 3 times, most recently from 4716ed8 to 6ad2300 Compare February 24, 2023 02:30
@0xhafa
Copy link
Copy Markdown
Collaborator Author

0xhafa commented Feb 24, 2023

Can you try to check and fix the failed test in CircleCI? 🙏

It didn't require to install foundry to fix the issue. There is a conflict happening when running slither with the foundry.toml file.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants