fix: remove redundant check on the v value

[ahramy]

AXE-4803

• Remove Unnecessary v value check in ERC20Permit#permit()
• The ERC20Permit contract implements an unnecessary check for the signature recovery ID (v) before calling the ecrecover function. This check is redundant as ecrecover inherently validates these values.
• The ecrecover function and the subsequent check for a valid recovered address are sufficient to ensure the signature's validity.
• Addressing it will optimize gas usage and simplify the contract code. This change aligns with best practices in Ethereum smart contract development, emphasizing efficiency and clarity in code implementation.

References:

• https://twitter.com/alexberegszaszi/status/1534461421454606336?s=20&t=H0Dv3ZT2bicx00hLWJk7Fg
• The latest Openzeppelin's ECDSA has removed the v check. by PR#3591

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.51%. Comparing base (be24c94) to head (130fcd0).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #279      +/-   ##
==========================================
- Coverage   98.51%   98.51%   -0.01%     
==========================================
  Files          19       19              
  Lines         608      607       -1     
  Branches      125      124       -1     
==========================================
- Hits          599      598       -1     
  Misses          5        5              
  Partials        4        4              

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

[ahramy]

Had discussion, and agreed that we don't want to change the InterchainToken bytecode, hence closing this PR.