search

curvefi / twocrypto-ng

Other
10 stars 6 forks source link

Unused declaration #3

Closed DmitryKorovyn closed 9 months ago

DmitryKorovyn commented 10 months ago

Please remove all occurrences of string @param k The index of the coin. from file contracts/CurveTwocryptoOptimized.vy: some versions of vyper-compiler are failing at compilation.

Compiling contracts...
  Vyper version: 0.3.10
CompilerError: vyper returned the following errors:

Method has no parameter 'k'
  line 1545:5
       1544          prices (dy/dx) calculated AFTER the latest trade.
  ---> 1545     @param k The index of the coin.
  ---------------^
       1546     @return uint256 Price oracle value of kth coin.

After removing there are no problems with compilation.

DmitryKorovyn commented 10 months ago

Made changes and openned pull request: https://github.com/curvefi/twocrypto-ng/pull/4