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

feat: remove asset holder allowlist (with latest main) #855

Closed
wants to merge 13 commits into from

test: fix a case where it still expected an event

abe55a1
Select commit
Loading
Failed to load commit list.
Closed

feat: remove asset holder allowlist (with latest main) #855

test: fix a case where it still expected an event
abe55a1
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 24, 2023 in 0s

99.52% of diff hit (target 91.57%)

View this Pull Request on Codecov

99.52% of diff hit (target 91.57%)

Annotations

Check warning on line 576 in contracts/discovery/GNS.sol

See this annotation in the file changed.

@codecov codecov / codecov/patch

contracts/discovery/GNS.sol#L576 

Added line #L576 was not covered by tests

Check warning on line 735 in contracts/discovery/GNS.sol

See this annotation in the file changed.

@codecov codecov / codecov/patch

contracts/discovery/GNS.sol#L735 

Added line #L735 was not covered by tests

Check warning on line 326 in contracts/staking/StakingExtension.sol

See this annotation in the file changed.

@codecov codecov / codecov/patch

contracts/staking/StakingExtension.sol#L326 

Added line #L326 was not covered by tests

Check warning on line 332 in contracts/staking/StakingExtension.sol

See this annotation in the file changed.

@codecov codecov / codecov/patch

contracts/staking/StakingExtension.sol#L332 

Added line #L332 was not covered by tests
View more details on Codecov