Lint/all

[tosettil-polimi]

Lint/all

This pull request includes several changes and fixes aimed at improving the code structure and consistency across different components related to transaction pool tasks.

Key updates include:

1. Refactoring and Consistency in Variable Naming:

   • Changed the variable duration_s to durationS in multiple files to ensure consistent naming conventions, particularly for configuration parameters that represent duration.

2. Improved Error Handling:

   • Modified the Execute function in tx_pool_clean/task.go to handle errors properly by checking for the result of t.cleanRecursive(client) and returning the error if one occurs, ensuring better error propagation and task failure handling.

3. Code Cleanup and Readability:

   • Refactored various instances of variable names such as task_ctx to taskCtx, improving readability and aligning with Go naming conventions.
   • Updated imports to maintain consistency, for example, changing package names like tx_load_tool to txloadtool.

4. Enhancements in Transaction Handling:

   • Modified the random selection of execution clients to utilize rand.Int(rand.Reader, big.NewInt(int64(len(executionClients))) instead of rand.Intn, ensuring better randomness and cryptographic safety.
   • Ensured proper handling of errors in various functions such as transaction execution and latency measurement.

5. File and Package Renaming:

   • Changed the package name for tx_pool_latency_analysis from txpool_latency_analysis to txpoollatencyanalysis for consistency across the project.

6. Minor Adjustments in Configuration Files:

   • Corrected the configuration in tx-pool-check.yaml and related files by replacing duration_s with durationS to match the new naming convention.

These changes should improve both the maintainability and functionality of the code, while also ensuring it adheres to consistent naming conventions and better error handling practices.