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(node/p2p)!: remove resource manager and it's API #1500

Merged
merged 4 commits into from
Dec 19, 2022

Conversation

Wondertan
Copy link
Member

This is a temporary fix for bootstrappers. Infinite Limits seem to be buggy. The proper fix is to update to the latest version of libp2p

This is a temporary fix for bootstrappers. Infinite Limits seem to be buggy. The proper fix is to update to the latest version of libp2p
@Wondertan Wondertan added kind:break! Attached to breaking PRs kind:fix Attached to bug-fixing PRs labels Dec 16, 2022
@Wondertan Wondertan self-assigned this Dec 16, 2022
@Wondertan
Copy link
Member Author

@distractedm1nd, do we need to run openrpc-gen? It errors out for me Proof error

vgonkivs
vgonkivs previously approved these changes Dec 16, 2022
@codecov-commenter
Copy link

codecov-commenter commented Dec 16, 2022

Codecov Report

Merging #1500 (364fcab) into main (3a58679) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1500      +/-   ##
==========================================
+ Coverage   55.40%   55.47%   +0.07%     
==========================================
  Files         203      203              
  Lines       12146    12125      -21     
==========================================
- Hits         6729     6726       -3     
+ Misses       4747     4731      -16     
+ Partials      670      668       -2     
Impacted Files Coverage Δ
nodebuilder/p2p/mocks/api.go 2.83% <ø> (+0.17%) ⬆️
nodebuilder/p2p/module.go 93.93% <ø> (+8.64%) ⬆️
nodebuilder/p2p/p2p.go 52.50% <ø> (+1.38%) ⬆️
nodebuilder/p2p/host.go 90.62% <100.00%> (-0.29%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@liamsi liamsi changed the title fix(node/p2p)!: remove resourse manager and it's API fix(node/p2p)!: remove resource manager and it's API Dec 16, 2022
@distractedm1nd
Copy link
Collaborator

@Wondertan No we don't need to run openrpc-gen, the error is coming from a missing example type and is unrelated

Copy link
Collaborator

@distractedm1nd distractedm1nd left a comment

Choose a reason for hiding this comment

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

please update ADR

Copy link
Collaborator

@distractedm1nd distractedm1nd left a comment

Choose a reason for hiding this comment

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

Didn't realize not removing from ADR was intentional

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:break! Attached to breaking PRs kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants