foundry-rs / foundry

Foundry is a blazing fast, portable and modular toolkit for Ethereum application development written in Rust.
https://getfoundry.sh
Apache License 2.0
8.33k stars 1.76k forks source link

Revert "fix(remappings): check if remapping to add starts with existing remapping name (#9246)" #9274

Closed grandizzy closed 2 weeks ago

grandizzy commented 2 weeks ago

This reverts commit 455ba9b1b736766232d84ba1790ac9ba6ca944de.

Motivation

Fix added in https://github.com/foundry-rs/foundry/pull/9246 affects existing projects compilation The changed code (checking remappings to add against existing) doesn't account scenario where @openzeppelin/contracts is already added and @openzeppelin/contracts-upgradeable remapping will be discarded (as remapping to add name starts with existing)

Revert and investigate failures Closes #9271 Closes #9272

Solution