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

fix: handle new peer by store #104

Merged
merged 1 commit into from
May 6, 2022
Merged

Conversation

gfanton
Copy link
Member

@gfanton gfanton commented May 6, 2022

No description provided.

@gfanton gfanton requested a review from a team as a code owner May 6, 2022 14:32
@gfanton gfanton self-assigned this May 6, 2022
@auto-add-label auto-add-label bot added the bug Something isn't working label May 6, 2022
@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label May 6, 2022
@codecov
Copy link

codecov bot commented May 6, 2022

Codecov Report

Merging #104 (3130869) into master (3130869) will not change coverage.
The diff coverage is n/a.

❗ Current head 3130869 differs from pull request most recent head 1077e23. Consider uploading reports for the commit 1077e23 to get more accurate results

@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   63.37%   63.37%           
=======================================
  Files          35       35           
  Lines        2482     2482           
=======================================
  Hits         1573     1573           
  Misses        625      625           
  Partials      284      284           
Flag Coverage Δ
unittests 63.37% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3130869...1077e23. Read the comment docs.

Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
@trafico-bot trafico-bot bot added ✅ Approved Feature request has been approved and removed 🔍 Ready for Review Pull Request is not reviewed yet labels May 6, 2022
@gfanton gfanton merged commit b99ebe6 into berty:master May 6, 2022
@gfanton gfanton deleted the fix/new-peer-event branch May 6, 2022 15:29
@trafico-bot trafico-bot bot added ✨ Merged and removed ✅ Approved Feature request has been approved labels May 6, 2022
@github-actions
Copy link

github-actions bot commented May 6, 2022

🎉 This PR is included in version 1.17.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants