Skip to content
This repository has been archived by the owner on Oct 19, 2024. It is now read-only.

refactor(abigen): inline docs #2090

Merged
merged 5 commits into from
Jan 31, 2023
Merged

Conversation

DaniPopes
Copy link
Collaborator

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Updated the changelog
  • Breaking changes

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gakonst
Copy link
Owner

gakonst commented Jan 30, 2023

error[E0599]: no function or associated item named `default` found for struct `seaport::BadContractSignature` in the current scope
   --> ethers-contract/tests/it/abigen.rs:649:73
    |
640 |     abigen!(Seaport, "./tests/solidity-contracts/seaport.json");
    |     ----------------------------------------------------------- function or associated item `default` not found for this struct
...
649 |     let err = SeaportErrors::BadContractSignature(BadContractSignature::default());
    |                                                                         ^^^^^^^ function or associated item not found in `seaport::BadContractSignature`
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following traits define an item `default`, perhaps you need to implement one of them:
            candidate #1: `std::default::Default`
            candidate #2: `tinyvec::array::Array`

Seems to have broken something here? @DaniPopes

@mattsse
Copy link
Collaborator

mattsse commented Jan 30, 2023

looks like all infura keys are completely exhausted -.-

having #2075 would def help

@gakonst gakonst merged commit fd02bbc into gakonst:master Jan 31, 2023
@DaniPopes DaniPopes deleted the refactor/abigen-docs branch February 17, 2023 06:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants