Remove orphan instance

[smelc]

Changelog

- description: |
    Remove orphan instance
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  - refactoring    # QoL changes
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

Follow-up of @carbolymer's remark: IntersectMBO/cardano-api#701 (comment)

How to trust this PR

It still compiles

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• Self-reviewed the diff

[Jimbo4350]

You have a build failure

[Jimbo4350]

Why not also implement:

instance Convert Era MaryEraOnwards where

We can avoid calling convert twice.

[carbolymer]

That was the original issue, wasn't it?

[smelc]

Yes this is what we want to avoid indeed 😄