Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ticket 737/ima switch toethers js #1410

Closed
wants to merge 252 commits into from

Conversation

sergiy-skalelabs
Copy link
Contributor

@sergiy-skalelabs sergiy-skalelabs commented Jan 20, 2023

  • IMA Agent was re-written 90% from scratch and now based on ethers.js
  • All JavaScript code is ES6 now, so all files are *.mjs
  • Fixed OKR requirements, all JS lines of code are limited to 100 characters, all JS functions are limited to 200 lines of code including empty lines, everything named camelCase everywhere it was possible, massive source code cleanup
  • Fixed documentation missmprints and errors, updated documentation with descriptions of changed command line options
  • Fixed PY tests run issues on fast local computers
  • Fixed JS tests errors running on local computers
  • Cleaned up docker file and github action scripts
  • Improved IMA Agent logging filters, all logging messages correspond to appropriate --verbose=0..9 level
  • Fixed all codacy issues and warnings, codacy report is completely clean green now

sergiy-skalelabs and others added 28 commits May 15, 2023 17:55
@PolinaKiporenko PolinaKiporenko deleted the ticket-737/IMA-switch-toethers-js branch December 6, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IMA - switch to ethers.js
3 participants