-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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.
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
feat: enhance the solidity test artifacts discovery #5829
base: v-next
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
hardhatTotal size of the bundle: List of dependencies (sorted by size)
|
0bf17ba
to
b3c8f2b
Compare
b3c8f2b
to
b4dacbc
Compare
This PR makes the check whether an artifact is a test artifact more accurate. It implements the design described in https://www.notion.so/nomicfoundation/Test-Artifacts-Discovery-11f578cdeaf580bdb7e5c4268c5f248d
The base cases of the newly added
isTestArtifact
method have been tested using unit tests and manually.It introduces 2 new dependencies:
@nomicfoundation/slang
- 2.6M - direct@bytecodealliance/preview2-shim
- 524K - transitive