Open m4sterbunny opened 3 months ago
@m4sterbunny Appreciate the posts.. Housekeeping feedback, I would request in the future you use issue templates to help guide the creation of an issue with more details, before posting a PR. Mixed opinions of #13200 and it can help to discuss before hand.
Left a note on the PR, but didn't agree with those changes proposed. Will leave this open though, would encourage you to provide more details or argue your case more and we can figure out a game plan.
The validator client, while typically bundled with the Consensus client, does not need to be run >> this is an active decision that also requires stake >> IMO it helps to clarify this early as the term Full node could otherwise be misleading (Full implies you run everything from a low-knowledge persepective).
Unidirectional arrow is to clear up the implication that EL can "drive" CL. CL makes calls to EL via the Engine API not vv. Yes, the EL responds to these requests >> This was from Besu team when I was attempting to understand further, so I am trusting my source. Alt would be dotted arrow back from EL.
WRT issue templates -- I did shop, found nothing that matched.
This issue is stale because it has been open 30 days with no activity.
docs(proof) needed to fix consistency of abrevs, reduce verboseness, improve clarity
I will take this one