fix: remove autominer

[gabriel-aranha-cw]

No description provided.

[github-actions]

PR Review 🔍

⏱️ Estimated effort to review [1-5]	2, because the PR involves a straightforward removal of a specific feature (auto-mining) from the code, which is a small change in terms of lines of code. However, understanding the implications of this change on the overall system might require a deeper understanding of the context in which the mining occurs.
🧪 Relevant tests	No
⚡ Possible issues	Possible Bug: Removing the automine feature without ensuring that there are alternative mechanisms in place for mining and committing blocks could lead to scenarios where transactions are not processed. This could be critical depending on the system's reliance on the automine feature.
🔒 Security concerns	No

Code feedback:

---

relevant file	src/eth/executor.rs
suggestion	Consider implementing a conditional feature or configuration option to enable or disable automining based on the environment or user preference. This would provide flexibility and maintain functionality for scenarios where automining is necessary. [important]
relevant line	- let miner = self.miner.lock().await;

---

[github-actions]

PR Code Suggestions ✨

Category	Suggestion	Score
Maintainability	Add error handling or logging after removing mining operations Ensure proper error handling or logging for the removed mining operations. Previously, errors from mining operations were propagated upwards. Consider adding error handling or logging to maintain robustness. src/eth/executor.rs [280] +// Consider adding error handling or logging here break tx_execution; Suggestion importance[1-10]: 7 Why: The suggestion is relevant as it addresses the removal of mining operations which might have had error handling embedded. Adding error handling or logging is important for maintaining the robustness of the system. However, the suggestion does not specify exactly how to implement this, which would have been more helpful.	7