Remove hardcode values related to voting governance

[JohnyVasamsetti]

Context

The GovernorSettings contract previously used hardcoded values for the following governance parameters:

• votingDelay: Delay before voting on a proposal starts (default: 1 block).
• votingPeriod: Duration for which voting remains open (default: 1 week).
• proposalThreshold: Minimum number of votes required to create a proposal (default: 0).

These values were directly embedded in the Governor contract and were also used in the GovernorTest contract for testing purposes. It's a bit confusion since we introduced the magic numbers. Removing magic numbers might help us to understand it in a better way.

Changes Introduced

This PR refactors the Governor contract to remove hardcoded governance parameter values and instead allows them to be passed as arguments during deployment.

The constructor has been updated to:
constructor( IVotes _token, TimelockController _timelock, uint48 _initialVotingDelay, uint32 _votingPeriod, uint256 _votingThreshold, uint256 _quorumPercentage )

Benifits

• Deployers can customize governance settings to suit their specific requirements without modifying the contract code.
• The updated design enables testing with different parameter values, improving the robustness of the contract.