🔧 chore: update Solidity dependencies

[Sabnock01]

Removes the now deprecated remappings for Uniswap in foundry.toml.

Although it looks like forge-std isn't used anywhere either, I suppose there may be future need for it.

Update: Also pushed a commit to fix the .gitmodules which was currently missing and modified foundry.toml to use the correct import style to match contracts/.

Resolves #541

[codecov]

Codecov Report

Merging #540 (cc6c256) into main (7c1fba4) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #540   +/-   ##
=======================================
  Coverage   58.75%   58.75%           
=======================================
  Files          11       11           
  Lines        3722     3722           
=======================================
  Hits         2187     2187           
  Misses       1535     1535           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[0xJepsen]

Shall we maybe close this in favor of #573 ?

[Sabnock01]

Shall we maybe close this in favor of #573 ?

That depends on how you plan to use the contracts. Do you want them to work out of the box? The two PR's don't really overlap. #573 is basically to just get rid of warnings / errors in the Solidity files. I'll leave to your discretion.

[0xJepsen]

Shall we maybe close this in favor of #573 ?

That depends on how you plan to use the contracts. Do you want them to work out of the box? The two PR's don't really overlap. #573 is basically to just get rid of warnings / errors in the Solidity files. I'll leave to your discretion.

No we never use these contracts directly at all. We just export their bindings with the crate. We might not even ever really need the foundry.toml in this repo either.

[Sabnock01]

Shall we maybe close this in favor of #573 ?

That depends on how you plan to use the contracts. Do you want them to work out of the box? The two PR's don't really overlap. #573 is basically to just get rid of warnings / errors in the Solidity files. I'll leave to your discretion.

No we never use these contracts directly at all. We just export their bindings with the crate. We might not even ever really need the foundry.toml in this repo either.

Fair enough. Will close.