mud icon indicating copy to clipboard operation
mud copied to clipboard

style: `forge fmt`

Open frolic opened this issue 2 years ago • 1 comments

At some point, prettier/vscode stopped talking to each other and wasn't getting format-on-save anymore.

This PR updates our vscode formatting to use forge fmt (and runs it across the repo).

Two notes that might block this PR from being good to merge:

  • it's tricky to add a root-level formatter for our git hooks because forge fmt uses the foundry.toml relative to where the command was run, rather than relative to the source file (see https://github.com/foundry-rs/foundry/issues/5686) but there's a workaround
  • we probably should use forge fmt for codegen too and remove the ignores, but needs some deeper code changes

frolic avatar Aug 21 '23 08:08 frolic

⚠️ No Changeset found

Latest commit: d1772e2731d43a86e0da98e4e15246e2abf9b962

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Aug 21 '23 08:08 changeset-bot[bot]