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

P2p peerdb tests #1573

Merged
merged 4 commits into from
May 9, 2024
Merged

Conversation

didaunesp
Copy link
Contributor

Set the overall coverage on peerdb module up to 83.3%

@didaunesp
Copy link
Contributor Author

@Djadih

p2p/peerManager/peerdb/counter_test.go Outdated Show resolved Hide resolved
p2p/peerManager/peerdb/datastore_test.go Outdated Show resolved Hide resolved
@didaunesp didaunesp force-pushed the p2p-peerdb-tests branch 3 times, most recently from 70daa78 to afb9623 Compare April 19, 2024 14:00
Djadih
Djadih previously approved these changes Apr 25, 2024
@didaunesp didaunesp changed the base branch from goldenage to main April 25, 2024 18:53
@didaunesp didaunesp dismissed Djadih’s stale review April 25, 2024 18:53

The base branch was changed.

wizeguyy
wizeguyy previously approved these changes May 8, 2024
Djadih
Djadih previously approved these changes May 8, 2024
@didaunesp didaunesp dismissed stale reviews from Djadih and wizeguyy via 09087ac May 9, 2024 16:20
@wizeguyy wizeguyy self-requested a review May 9, 2024 16:24
@didaunesp didaunesp merged commit 09087ac into dominant-strategies:main May 9, 2024
7 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.

4 participants