Refactor errors and log statements

[waymobetta]

Description

I noticed while reviewing the codebase that some errors being returned as well as log statements were written in such a way that was excessive.

Example: log.Info().Msg(fmt.Sprintf("Syncing validators from %s block", block.String()))

This could be rewritten as..

log.Info().Msgf("Syncing validators from %s block", block.String())

This refactoring also removes the need for the errors package to be imported in a few cases.

Related Issue Or Context

Closes: N/A

How Has This Been Tested? Testing details.

N/A

Types of changes

• [X] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)
• [ ] Documentation

Checklist:

• [ ] I have commented my code, particularly in hard-to-understand areas.
• [X] I have ensured that all acceptance criteria (or expected behavior) from issue are met
• [ ] I have updated the documentation locally and in chainbridge-docs.
• [ ] I have added tests to cover my changes.
• [ ] I have ensured that all the checks are passing and green, I've signed the CLA bot

[github-actions[bot]]

Go Test coverage is 40.7 %\ :sparkles: :sparkles: :sparkles:

[github-actions[bot]]

Go Test coverage is 40.7 %\ :sparkles: :sparkles: :sparkles: