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.

Sign up for GitHub

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

Jump to bottom

Deploy script for the protocol and plugin upgrades #384

Merged
merged 138 commits into from
May 24, 2023
Merged

Deploy script for the protocol and plugin upgrades #384

merged 138 commits into from
May 24, 2023

Conversation

heueristik
Copy link
Contributor

@heueristik heueristik commented May 22, 2023 •
edited
Loading

Description

Note, that @mathewmeconry tested the previous deploy scripts with tenderly and that tests are worked on by @Rekard0 in Task OS-441.

Task: OS-416

Type of change

  • deployment preparation

heueristik and others added 30 commits April 17, 2023 11:24
@heueristik
feat: protect the DAO executor against reentrancy
dd7a0dc
@heueristik
fix: remove console.log
ad2fd40
@heueristik
fix: prevent storage corruption
616b8bf
@heueristik
feat: refactored reentrancy guard into a modifier
e45854c
@heueristik
feat: added tests and preliminary versioning
2ac2262
@heueristik
chore: cleaning
9fb0473
@heueristik
fix: remove empty comment
a025523
@heueristik
fix: refactor findEventTopicLog function
3fcc083
@heueristik
fix: moved implementation slot and renamed file
02711cc
@heueristik
fix: remove redundant includes
360079c
@Rekard0
prepare package
1b14d42
@Rekard0
remember the branch
9d73698
@Rekard0
only artifact src
fe9fe0f
@Rekard0
correct the paths
c8f10de
@Rekard0
correct root path
c878bf4
@Rekard0
update hashes
f319e0c
@Rekard0
add logs
3575258
@Rekard0
only copy typechain
1c6b6f8
@Rekard0
generate typechain
9f961cc
@Rekard0
setup rollup
e8d6590
@Rekard0
update npm packages
930b436
@heueristik
Merge branch 'develop' into feature/OS-358-nonreentrant-executor
049ecc6
@Rekard0
update rollup index
5db940f
@Rekard0
add esm cjs
4dc131e
@Rekard0
add readme
6005f06
@Rekard0
remove gitIgnore
748b382
@heueristik
Merge branch 'f/OS-380_npm_package_versions' into feature/OS-358-nonr…
f78ad2c 
…eentrant-executor
@Rekard0
update typechain
f9858c7
@Rekard0
update typescript
1653ca1
@Rekard0
convert script to typescript
03ad311
@heueristik
fix: use deploy method
0129db0 
fix: use deploy method
@heueristik heueristik force-pushed the feature/OS-416-deploy-script branch from 2b03726 to 0129db0 Compare May 22, 2023 12:41
@heueristik heueristik mentioned this pull request May 24, 2023
Merged
12 tasks
@heueristik heueristik requested a review from brickpop May 24, 2023 09:47
brickpop
brickpop previously approved these changes May 24, 2023
View reviewed changes
@heueristik heueristik requested a review from brickpop May 24, 2023 12:37
Rekard0
Rekard0 reviewed May 24, 2023
View reviewed changes
Copy link
Contributor

@Rekard0 Rekard0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could u add a tag to all files, expressing the update
something like update_to_1_3_0 or what ever tag u prefer

Rekard0
Rekard0 approved these changes May 24, 2023
View reviewed changes
Copy link
Contributor

@Rekard0 Rekard0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@heueristik heueristik merged commit 43a5f39 into develop May 24, 2023
@heueristik heueristik deleted the feature/OS-416-deploy-script branch May 24, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rekard0 Rekard0 Rekard0 approved these changes

@brickpop brickpop brickpop approved these changes

@mathewmeconry mathewmeconry Awaiting requested review from mathewmeconry

Assignees
No one assigned
Labels
contracts
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@heueristik @brickpop @Rekard0