Forge fmt

[cwang25]

• Add forge fmt --check to action
• All changes are done by forge fmt

[scott-silver]

Add forge fmt --check to action

I added forge fmt --check to the Github Action that runs the unit tests, so now CI should fail if we have non-compliant code

[cwang25]

I purposely put it into lint.yml as a separate CI flows. So it's easier to see if it's failing at lint or failing at test

Before PR merge the workflow won't show (sadly that's how github works :( )
But once merged the lint.yml should appear as separate checks in PR interface.

I sort of prefer to have two checks in interface one lint one test How do you think?

[scott-silver]

that's fine with me, but I don't think lint.yml made it into this PR

[cwang25]