fix: remove anvil from inspect command and add support for target string #364
Conversation
☁️ Nx Cloud ReportCI is running/has finished running commands for commit d3a6c17. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
FuzzB0t
changed the title
packages/builder/src/schemas.zod.ts
Outdated
| !!val.match(artifactNameRegex) || | ||
| !!val.match(artifactPathRegex), | ||
| (val) => ({ | ||
| message: `"${val}" must be a valid ethereum address, previously defined contract step name, contract artifact name or filepath`, |
There was a problem hiding this comment.
previously defined contract step name => existing contract step name
its not necessary that the contract step was "previous" (although it must be depended upon) (soon we are getting rid of depends so probably just best to keep response simple)
Yeah this PR contains both changes, although somewhat irrelevant to eachother, I'll change the PR title to match the contents better. |
FuzzB0t
changed the title
No description provided.