Some incredibility checks were in the process_transaction function instead of the check_transaction function
Related to Issue #799
How was it fixed?
The following changes have been made in the PR
Change the check_transaction function to include all the includibility checks for a transaction
Get rid of the validate_transaction function and incorporate it within the check_transaction function.
Change the order of the check_transaction function inputs to reflect the chronological order of the forks in which the parameters were introduced.
Update the check_transaction function of the t8n tool work with the new signature of check_transaction for cancun. Note: This is a temporary change and the final version of this function can only be implemented after the changes have been ported over to the other forks.
Replace the use of ensure with explicit if statements.
What was wrong?
Some incredibility checks were in the
process_transactionfunction instead of thecheck_transactionfunctionRelated to Issue #799
How was it fixed?
The following changes have been made in the PR
check_transactionfunction to include all the includibility checks for a transactionvalidate_transactionfunction and incorporate it within thecheck_transactionfunction.check_transactionfunction inputs to reflect the chronological order of the forks in which the parameters were introduced.check_transactionfunction of the t8n tool work with the new signature of check_transaction forcancun. Note: This is a temporary change and the final version of this function can only be implemented after the changes have been ported over to the other forks.ensurewith explicit if statements.Cute Animal Picture