Add splitTestsCompilation solidity setting (1): Configuration updates#8128
Conversation
|
There was a problem hiding this comment.
Pull request overview
Adds a new splitTestsCompilation Solidity setting to Hardhat’s Solidity plugin configuration surface, including user-config type extensions, config schema validation, and resolved-config shaping, with accompanying test updates.
Changes:
- Extend Solidity config typings to include
splitTestsCompilation(user + resolved config). - Validate
splitTestsCompilationas an optional boolean across supported Solidity config shapes. - Resolve
splitTestsCompilationinto the finalsolidityconfig with a default offalse, and update tests/fixtures accordingly.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/hardhat/test/internal/builtin-plugins/solidity/config.ts | Adds validation + resolution test coverage for splitTestsCompilation, and updates resolved-config test fixture. |
| packages/hardhat/test/internal/builtin-plugins/solidity/build-system/build-system.ts | Updates SolidityConfig test fixture to include required splitTestsCompilation. |
| packages/hardhat/src/internal/builtin-plugins/solidity/type-extensions.ts | Extends CommonSolidityUserConfig and resolved SolidityConfig types with splitTestsCompilation. |
| packages/hardhat/src/internal/builtin-plugins/solidity/config.ts | Adds Zod schema validation for splitTestsCompilation and resolves it into SolidityConfig with default false. |
ab18c17 to
be8bf71
Compare
This PR introduces the new config, it's validation, and resolution