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

feat: deploy libraries and proxy contract as singletons #164

Closed
wants to merge 26 commits into from

Conversation

guidiaz
Copy link
Collaborator

@guidiaz guidiaz commented May 12, 2021

  • contracts: new SingletonFactory contract
  • contracts: refactored WitnetRequestBoardProxy contract
  • contracts: rearranged library contracts, Migrations and SingletonFactory into subfolders
  • migrations: new singletons.json configuration file
  • migrations: new *singleton*.js scripts for deploying as singletons the libraries and contracts referred on singletons.json
  • migrations: made initial migration verify existence of addresses.json file, and recreate file template if not found
  • migrations: rearranged migrations scripts, considering WitnetRequestBoard is not to become singleton
  • migrations: automatically update addresses.json if any change on singleton addresses is produced (i.e. contract codehash changed)
  • utils: new helper scripts (javascript)
  • test: some refactoring required as consider WRBProxy as singleton, and due to the change on the constructor

Closes #162

utils/traceEvents.js Outdated Show resolved Hide resolved
test/using_witnet.js Outdated Show resolved Hide resolved
test/using_witnet.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
@mariocao
Copy link
Contributor

Once the PR #165 is merged, yarn fmt will now check all javascript files for lint warnings and errors.

@mariocao
Copy link
Contributor

The PR is now merged, so after a rebase you will found "some" new warnings 😄

utils/utils.js Outdated Show resolved Hide resolved
utils/utils.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
migrations/04_singleton_proxy.js Outdated Show resolved Hide resolved
migrations/99_upgrade_proxy.js Outdated Show resolved Hide resolved
utils/traceEvents.js Show resolved Hide resolved
utils/utils.js Outdated Show resolved Hide resolved
migrations/02_singleton_libs.js Show resolved Hide resolved
migrations/02_singleton_libs.js Outdated Show resolved Hide resolved
migrations/01_initial.js Outdated Show resolved Hide resolved
@guidiaz
Copy link
Collaborator Author

guidiaz commented May 18, 2021

The PR is now merged, so after a rebase you will found "some" new warnings 😄

All warnings/errors solved on latest commit: 8c2cbdc

@guidiaz guidiaz requested review from aesedepece and mariocao May 19, 2021 10:11
@mariocao mariocao force-pushed the master branch 2 times, most recently from 2230f68 to f45186b Compare May 21, 2021 10:18
@guidiaz
Copy link
Collaborator Author

guidiaz commented Mar 22, 2023

Supeseded by #317.

@guidiaz guidiaz closed this Mar 22, 2023
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.

Use Singleton Factory (ERC-2470) for libraries and proxy
4 participants