Checking PC validity performance improvement

[stana-miric]

Description

This PR include changes related to validation of PreparedCertificate messages. Earlier there were separate functions for checking height,round,senders and proposal hash. Now they are merged into AreValidPCMessages to avoid iterating 5 times instead of once. We decided to do this because the code will still remain readable.

Changes include

• Bugfix (non-breaking change that solves an issue)
• Hotfix (change that solves an urgent issue, and requires immediate attention)
• New feature (non-breaking change that adds functionality)
• Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

• I have assigned this PR to myself
• I have added at least 1 reviewer
• I have added the relevant labels
• I have added sufficient documentation in code

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 88.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 89.48%. Comparing base (628065c) to head (43b9a86).
Report is 7 commits behind head on main.

Files	Patch %	Lines
messages/helpers.go	87.50%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   88.53%   89.48%   +0.94%     
==========================================
  Files           7        7              
  Lines        1527     1502      -25     
==========================================
- Hits         1352     1344       -8     
+ Misses        134      119      -15     
+ Partials       41       39       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Stefan-Ethernal]

Generally LGTM

[vcastellm]

LGTM neat