Skip to content

👷‍♂️ Snap, Lint #400

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jtriley2p wants to merge 4 commits into
base: main
Choose a base branch
from
Open

👷‍♂️ Snap, Lint #400

jtriley2p wants to merge 4 commits into from

Conversation

@jtriley2p
Copy link
Contributor

@jtriley2p jtriley2p commented Nov 22, 2023

Description

forge snapshot to fix the snapshot for ci --check

npm run lint to format src/test/CREATE3.t.sol

Closes #398

Checklist

  • Ran forge snapshot?
  • Ran npm run lint?
  • Ran forge test?
@jtriley2p
Copy link
Contributor Author

jtriley2p commented Nov 22, 2023

W H A T
@jtriley2p jtriley2p changed the title 👷‍♂️ Snap, LInt Nov 22, 2023
@jtriley2p
Copy link
Contributor Author

jtriley2p commented Nov 22, 2023

not sure how to reproduce, forge snapshot --check doesn't fail locally
@0xCLARITY
Copy link

0xCLARITY commented Nov 22, 2023

Pretty sure you're running into this: foundry-rs/foundry#5199
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@jtriley2p @0xCLARITY