feat(primitives): remove mod ethers_compat

[tcoratger]

Should close #4738.

[tcoratger]

This is just a draft here, I'm not that familiar with alloy but I think that we need to change GethInstance in:

reth/crates/net/network/tests/it/clique/geth.rs

Lines 36 to 41 in 272f062

	pub struct CliqueGethInstance {
	/// The spawned [`GethInstance`](ethers_core::utils::GethInstance).
	pub instance: GethInstance,
	/// The provider who can talk to this instance
	pub provider: SignerMiddleware<Provider<Ws>, Wallet<SigningKey>>,
	}

in order to remove the ethers_core dependency here. Maybe this is is part of #4737 or to be included here after the merge?

[DaniPopes]

Ah they're used for tests. This is fine for now, you can make the modules #[cfg(feature = "test-utils")]

[codecov]

Codecov Report

Merging #4771 (f225336) into main (272f062) will decrease coverage by 0.19%.
Report is 19 commits behind head on main.
The diff coverage is n/a.

Files	Coverage Δ
crates/primitives/src/genesis.rs	97.38% <ø> (+0.02%)	⬆️
crates/primitives/src/header.rs	94.88% <ø> (ø)

... and 34 files with indirect coverage changes

Flag	Coverage Δ
integration-tests	16.18% <ø> (-0.34%)	⬇️
unit-tests	63.27% <ø> (-0.16%)	⬇️

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

Components	Coverage Δ
reth binary	32.11% <ø> (+0.03%)	⬆️
blockchain tree	83.69% <ø> (ø)
pipeline	88.45% <ø> (-0.09%)	⬇️
storage (db)	73.47% <ø> (-0.02%)	⬇️
trie	94.52% <ø> (+0.03%)	⬆️
txpool	49.51% <ø> (ø)
networking	76.47% <ø> (-0.59%)	⬇️
rpc	57.71% <ø> (+0.04%)	⬆️
consensus	62.59% <ø> (-0.26%)	⬇️
revm	28.33% <ø> (ø)
payload builder	8.20% <ø> (ø)
primitives	86.37% <ø> (-0.17%)	⬇️

[tcoratger]

Ah they're used for tests. This is fine for now, you can make the modules #[cfg(feature = "test-utils")]

@DaniPopes Done.