Closed bryanchriswhite closed 11 months ago
Everything looks good to me, I saw the comments about potentially unrelated hunks. Will they remain in this PR or move?
I think I force pushed over the commits that I referenced, my apologies. The changes I was referencing are still in this branch. I was just trying to point out that parts of those changes didn't necessarily fit with the rest.
Linking the mindmap from this comment in #844 for additional context.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
Our GitHub checks need improvements? Share your feedbacks!
LGTM +1 Replacing providers with submodules seems worth an ADR, if you want help writing that up let's take a look at it @bryanchriswhite
@dylanlott it's more like the providers need to become submodules.
:rotating_light: Please don't delete this branch! I will do so when all downstream branches have been rebased, thank you! :rotating_light:
Description
Introduces the
Submodule
interface and applies it to the P2P module's peerstore provider.Issue
Deliverables 1, 2, 3, & 5:
810
Type of change
Please mark the relevant option(s):
List of changes
IntegratableModule
toIntegrableModule
InitializableModule
toInjectableModule
InjectableModule#Create()
asMoudle
also embedsModuleFactoryWithConfig
Submodule
interface typeTesting
make develop_test
; if any code changes were mademake test_e2e
on k8s LocalNet; if any code changes were madee2e-devnet-test
passes tests on DevNet; if any code was changedRequired Checklist
godoc
format comments on touched members (see: tip.golang.org/doc/comment)If Applicable Checklist
shared/docs/*
if I updatedshared/*
README(s)