search

LimeChain / etherlime

Dapp Development framework based on ethers.js
MIT License
185 stars 41 forks source link

Replace global `utils` variable with `etherlime.utils` #267

Open Daniel-K-Ivanov opened 5 years ago

Daniel-K-Ivanov commented 5 years ago

Generally speaking utils global variable is not okay, because every project has its own utils file.

To add on top of that - it is somewhat confusing. I personally did not expected to have global utils variable.

Suggestion: replace the current utils global variable with etherlime.utils global variable.

Example usage: Before:

utils.hasEvent(transactionReceipt, LimeFactory, expectedEvent);

Suggestion:

etherlime.utils.hasEvent(transactionReceipt, LimeFactory, expectedEvent);