feat: Remove associated type consensus error

[JoshdfG]

Closes #519

[codecov]

Codecov Report

❌ Patch coverage is 51.72414% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.22%. Comparing base (957e70c) to head (08c7738).
⚠️ Report is 3 commits behind head on unstable.

Files with missing lines	Patch %	Lines
crates/net/downloaders/src/file_client.rs	0.00%	3 Missing ⚠️
crates/cli/commands/src/stage/dump/execution.rs	0.00%	2 Missing ⚠️
crates/cli/commands/src/stage/dump/merkle.rs	0.00%	2 Missing ⚠️
crates/consensus/consensus/src/noop.rs	0.00%	2 Missing ⚠️
crates/cli/commands/src/import_core.rs	0.00%	1 Missing ⚠️
crates/net/downloaders/src/bodies/queue.rs	0.00%	1 Missing ⚠️
crates/net/downloaders/src/bodies/request.rs	0.00%	1 Missing ⚠️
crates/rpc/rpc/src/validation.rs	0.00%	1 Missing ⚠️
crates/stages/stages/src/stages/execution.rs	66.66%	1 Missing ⚠️

Additional details and impacted files

Files with missing lines	Coverage Δ
crates/consensus/consensus/src/lib.rs	0.00% <ø> (ø)
crates/engine/service/src/service.rs	100.00% <100.00%> (ø)
crates/engine/tree/src/download.rs	21.25% <100.00%> (ø)
crates/engine/tree/src/tree/mod.rs	50.51% <100.00%> (+0.06%)	⬆️
crates/engine/tree/src/tree/payload_validator.rs	66.11% <100.00%> (ø)
crates/net/downloaders/src/bodies/bodies.rs	28.57% <100.00%> (ø)
crates/net/downloaders/src/bodies/task.rs	42.85% <ø> (ø)
crates/net/p2p/src/full_block.rs	0.98% <100.00%> (-0.97%)	⬇️
crates/node/builder/src/components/builder.rs	54.88% <ø> (ø)
crates/node/builder/src/components/consensus.rs	0.00% <ø> (ø)
... and 15 more

... and 36 files with indirect coverage changes

Flag	Coverage Δ
e2e	35.67% <51.72%> (+0.20%)	⬆️
unit	26.73% <0.00%> (+<0.01%)	⬆️

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

Components	Coverage Δ
reth binary	53.14% <ø> (ø)
op historical proof	89.97% <ø> (+0.03%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[emhane]

a bunch of unrelated semicolon additions need to be reverted, otherwise lgtm

[emhane]

pls remove all the random semicolons that were added in this pr. viewing the Files Changed of the pr in the browser in Github helps to see the diff clearer, and should be possible to identify all the occurrences of this random added semicolon and remove them.

[JoshdfG]

pls remove all the random semicolons that were added in this pr. viewing the Files Changed of the pr in the browser in Github helps to see the diff clearer, and should be possible to identify all the occurrences of this random added semicolon and remove them.

It has all been fixed, sorry about that, the cargo +nightly fmt didn't work had to do it manually from the browser.

[emhane]

pls remove all the random semicolons that were added in this pr. viewing the Files Changed of the pr in the browser in Github helps to see the diff clearer, and should be possible to identify all the occurrences of this random added semicolon and remove them.

It has all been fixed, sorry about that, the cargo +nightly fmt didn't work had to do it manually from the browser.

$ rustup toolchain install nightly