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: some refactoring #11

Merged
merged 13 commits into from
Jun 30, 2023
Merged

Conversation

Foivos
Copy link
Contributor

@Foivos Foivos commented Jun 29, 2023

No description provided.

@milapsheth milapsheth marked this pull request as ready for review June 29, 2023 16:22
@Foivos Foivos changed the base branch from feat/deploy-create3-deployer to feat/deploy-governance June 29, 2023 16:28
Base automatically changed from feat/deploy-governance to feat/deploy-create3-deployer June 29, 2023 17:24
@deanamiel deanamiel merged commit f63bf92 into feat/deploy-create3-deployer Jun 30, 2023
@deanamiel deanamiel deleted the feat/some-refactoring branch June 30, 2023 00:09
deanamiel added a commit that referenced this pull request Jul 3, 2023
* first iteration of create3 deployer deployment script

* some minor changes

* feat: deployment script for governance, mint limiter, and operators (#9)

* deployment scripts for non-upgradable contracts using create3

* fixed linting errors

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>

* feat: some refactoring (#11)

* deployment scripts for non-upgradable contracts using create3

* fixed linting errors

* refactoring verification to make it a bit cleaner.

* prettier

* prettier

* fix small erro

* fix some errors

* remove unused file

* prettier

* Added fixes for non-upgradable as well

* fixed bugs

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>
Co-authored-by: Dean <deanamiel1@gmail.com>

* removed duplicate deployCreate3 function

* potential lint fix

* addressed comments

* addressed comments

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>
Co-authored-by: Foivos <foivos@axelar.network>
Co-authored-by: Milap Sheth <milap@axelar.network>
milapsheth added a commit that referenced this pull request Jul 18, 2023
* first iteration of create3 deployer deployment script

* some minor changes

* refactoring verification to make it a bit cleaner.

* feat: deployment script for governance, mint limiter, and operators (#9)

* deployment scripts for non-upgradable contracts using create3

* fixed linting errors

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>

* stash

* feat: some refactoring (#11)

* deployment scripts for non-upgradable contracts using create3

* fixed linting errors

* refactoring verification to make it a bit cleaner.

* prettier

* prettier

* fix small erro

* fix some errors

* remove unused file

* prettier

* Added fixes for non-upgradable as well

* fixed bugs

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>
Co-authored-by: Dean <deanamiel1@gmail.com>

* removed duplicate deployCreate3 function

* potential lint fix

* added an its deployment script

* can test localy easily

* done

* deployed on avalanche as well.

* made lint happy

* deploy its now returns the its

* not using hardhat in the its deployment script

* added a postinstall

* using the new name for linker router

* importing artifacts from contracts

* deployed the newer version to some testnets.

* prettier

* prettier

* README

* minor fixes

* name change

* fixed lint errors

* addressed comments

* update deps

* use 0.8.19

* use more runs

* deployed mostly everywhere

* deployed on binance

* stash

* added an upgrade function

* fix merge issues

* lint

* prettier

* update its

* readme

* send token script

* remove aurora and filecoin

* fix bug

* add balance checker script and update send token

* fix hardhat config

* create3 deployer improvements

* its deployments

* script improvements

* lint

* update readme

---------

Co-authored-by: Dean Amiel <damiel@deans-mbp.lan>
Co-authored-by: Dean <deanamiel1@gmail.com>
Co-authored-by: Milap Sheth <milap@axelar.network>
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.

3 participants