Move EIP-712 to Final

[frangio]

There have been no comments since Last Call was announced.

[eth-bot]

A critical exception has occurred:
Message: pr 5433 is already merged; quitting
(cc @alita-moore, @mryalamanchi)

[Pandapip1]

The formatting is a bit wonky, but otherwise LGTM. Since this is a pretty significant EIP, I'm going to hold off from flat-out approving it and will wait for someone else to also signal approval.

[frangio]

@Pandapip1 Do you mean the images? I agree but I'm not sure what's the right way to resize them in markdown that will work on the final rendered site. Does anyone else know?

Could also just resize the images and reupload them.

[Pandapip1]

Not the images; the spacing in between headers is inconsistent, as well as a few other things. I don't think they result in any issues. No action is needed on your part.

[Pandapip1]

Please close and re-open this PR to merge it.

[frangio]

@Pandapip1 Should I do that before or after author approval? (Note: I'm not author)

[Pandapip1]

It has been approved.

[Pandapip1]

CC @recmo, @LogvinovLeon, @expede

[wschwab]

I only noticed that 712 made Last Call now - I had originally written a resolution to the checklist that originally was put in the Security Considerations, which I think is removed in this version of 712. Would I be able to put it back in?

Reference: #4383

[frangio]

@wschwab We discussed that section in #5271 (comment) with the editors and agreed that it felt out of place in the EIP as it was more of an implementation log.