feat: Remove duplicated to method in TransactionResponse

[moricho]

Motivation

TransactionResponse's base trait alloy_consensus::Transaction already has to method with the exact same default implementation.
It causes the following error when using TransactionResponse along with Transaction, and the result is the same when to() is called with either of the above traits. It would be a bit confusing.

error[E0034]: multiple applicable items in scope
   --> crates/.../...
    |
216 |                 .to()
    |                  ^^ multiple `to` found
    |
    = note: candidate #1 is defined in an impl of the trait `TransactionResponse` for the type `alloy_rpc_types_eth::Transaction<T>`
    = note: candidate #2 is defined in an impl of the trait `TransactionTrait` for the type `alloy_rpc_types_eth::Transaction<T>`
help: disambiguate the method for candidate #1
    |
215 |             && TransactionResponse::to(&tx)
    |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: disambiguate the method for candidate #2
    |
215 |             && TransactionTrait::to(&tx)
    |                ~~~~~~~~~~~~~~~~~~~~~~~~~

Solution

PR Checklist

• Added Tests
• Added Documentation
• Breaking changes

[mattsse]

ty, yep this should be removed