This repository was archived by the owner on Aug 14, 2025. It is now read-only.

Commit 52186a5

authored

Merge pull request #18 from ItsNickBarry/path-fix

fix sources path resolution for compatibility with hardhat-foundry

2 parents 83328c5 + 8857f28 commit 52186a5Copy full SHA for 52186a5

File tree

-3

lines changed

-3

lines changed

Lines changed: 6 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -19,14 +19,17 @@ const generate = function (dependency) {`
`19`	`19`	`task(TASK_COMPILE, async function (args, hre, runSuper) {`
`20`	`20`	`const config = hre.config.dependencyCompiler;`
`21`	`21`
`22`		`- const directory = path.resolve(hre.config.paths.sources, config.path);`
	`22`	`+ // sources path must be "resolved" for compatibility with @nomicfoundation/hardhat-foundry package`
	`23`	`+ const sources = path.resolve(hre.config.paths.sources);`
	`24`	`+`
	`25`	`+ const directory = path.resolve(sources, config.path);`
`23`	`26`	const tracker = path.resolve(directory, `.${ name }`);
`24`	`27`
`25`		`- if (!directory.startsWith(hre.config.paths.sources)) {`
	`28`	`+ if (!directory.startsWith(sources)) {`
`26`	`29`	`throw new HardhatPluginError('resolved path must be inside of sources directory');`
`27`	`30`	`}`
`28`	`31`
`29`		`- if (directory === hre.config.paths.sources) {`
	`32`	`+ if (directory === sources) {`
`30`	`33`	`throw new HardhatPluginError('resolved path must not be sources directory');`
`31`	`34`	`}`
`32`	`35`

Comments

(0)