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

chore(dot/network): added common request-response protocol #3334

Merged
merged 12 commits into from
Jul 4, 2023

Conversation

kishansagathiya
Copy link
Contributor

Changes

Tests

go test -tags integration github.com/ChainSafe/gossamer

Issues

Primary Reviewer

@EclesioMeloJunior

@kishansagathiya kishansagathiya marked this pull request as ready for review June 20, 2023 08:51
@kishansagathiya kishansagathiya changed the title Added common request-response protocol chore(dot/network): Added common request-response protocol Jun 20, 2023
@kishansagathiya kishansagathiya changed the title chore(dot/network): Added common request-response protocol chore(dot/network): added common request-response protocol Jun 20, 2023
dot/network/request_response.go Outdated Show resolved Hide resolved
dot/network/request_response.go Outdated Show resolved Hide resolved
dot/network/request_response.go Outdated Show resolved Hide resolved
Copy link
Member

@EclesioMeloJunior EclesioMeloJunior left a comment

Choose a reason for hiding this comment

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

Nice work so far!! Will provide more freedom to use the network layer, just few points

dot/network/service.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jun 23, 2023

Codecov Report

Merging #3334 (b3a6c02) into development (457fe39) will decrease coverage by 0.08%.
The diff coverage is 13.69%.

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #3334      +/-   ##
===============================================
- Coverage        51.52%   51.44%   -0.08%     
===============================================
  Files              224      225       +1     
  Lines            28111    28117       +6     
===============================================
- Hits             14484    14465      -19     
- Misses           12242    12267      +25     
  Partials          1385     1385              

Copy link
Contributor

@jimjbrettj jimjbrettj left a comment

Choose a reason for hiding this comment

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

looks very nice, great stuff!

@kishansagathiya kishansagathiya merged commit ac52090 into development Jul 4, 2023
@kishansagathiya kishansagathiya deleted the kishan/feat/parachain/request-response-2 branch July 4, 2023 13:28
Copy link

🎉 This PR is included in version 0.8.0 🎉

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants