search

entropyxyz / sdk

Official JavaScript SDK for Entropy blockchain.
GNU Affero General Public License v3.0
7 stars 0 forks source link

Jawn/remove any #216

Closed jawndiego closed 10 months ago

jawndiego commented 10 months ago

resolves #206

added explicit types to resolve yarn lint warnings. Please double check that res in crypto still works like we intend please!

there is one "any" flag left in signing:


  [key: string]: any
}```
frankiebee commented 10 months ago

resolves #206

added explicit types to resolve yarn lint warnings. Please double check that res in crypto still works like we intend please!

there is one "any" flag left in signing:

  [key: string]: any
}```

oh lol i was naught and didnt read this before review

jawndiego commented 10 months ago

now all any's are now removed. TxParams is now typed as Transaction | unknown.

tests are passing with just Transaction type, but we may want unknown in preparation to accommodate new adapters.