Add supported-chains cmd & Improve Error Messages#330
Add supported-chains cmd & Improve Error Messages#330
Conversation
|
👋 ejacquier, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
| target = "(none)" | ||
| } | ||
| return Inputs{}, fmt.Errorf( | ||
| "no RPC URLs found for target %q\n\n"+ |
There was a problem hiding this comment.
Don't we have a generalised rpc check in the root of the cmd package?
There was a problem hiding this comment.
Not quite. cmd/root.go has isLoadDeploymentRPC() which decides whether to load RPC settings (for deploy/pause/activate/delete). That's a
config-loading gate, not a validation error.
a83126d to
45e16f6
Compare
The merge-base changed after approval.
|
| Version | Value |
|---|---|
| Current Fork | v1.17.0 |
| Latest Upstream | null |
Action Required
- Review abigen changes in upstream (only the
accounts/abi/binddirectory matters) - Compare with our fork in
cmd/generate-bindings/bindings/abigen/ - If relevant changes exist, sync them and update
FORK_METADATA.md - If no abigen changes, just update the version in
FORK_METADATA.mdtonull
Files to Review
cmd/generate-bindings/bindings/abigen/bind.gocmd/generate-bindings/bindings/abigen/bindv2.gocmd/generate-bindings/bindings/abigen/template.go
cc @smartcontractkit/bix-framework
Supported Chains
cre workflow supported-chainscommand (sorted alphabetically)📊 Error Handling Improvements
cre init,--project-root, orcdinto project@chainlink/cre-sdk+ suggestsbun install/cre updaterpcssection and--supported-chainscre workflow initor checking path