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/mode upgrade v2 #61

Merged
merged 91 commits into from
Feb 25, 2025
Merged

Deploy/mode upgrade v2 #61

merged 91 commits into from
Feb 25, 2025

Conversation

jordaniza
Copy link
Collaborator

No description provided.

brickpop and others added 30 commits October 29, 2024 11:33
@brickpop
Refactoring the makefile to make targets generic
5ff4451
@brickpop
Refactoring the Makefile, optimizing env vars and making the E2E test…
db1cfb7 
… use AragonTest
@brickpop
Making the makefile pick the right values depending on the target
2376427
@brickpop
Keeping the output in a deployment log file
c5e1259
@brickpop
Improved readability, adding E2E tests to use an existing token
ec49783
@brickpop
Final touches
ea3f69f
@brickpop
Moving rule based values next to their usage
85ed996
@brickpop
Renaming the forking test modes
c6423d2
@brickpop
Final touches
f42adf6
@brickpop
Extended deployment checklist
7d28eb7
@brickpop
Adding a refund target
6b9f13d
@brickpop
Minor edits
15ee37e
@brickpop
Minor
232af8a
@brickpop
Minor
73c5a13
@brickpop
Ready to merge
e9fa39b
@brickpop
Assign
f87f6e2
@brickpop
Merge pull request #37 from aragon/f/static-env-file
5dcc440 
Static env files
@brickpop
Improved deployment checklist
ae21ce9
@xavikh
Add ERC721 metadata
a0a13b3
@xavikh
Add BaseURISet event
ee6e24c
@xavikh
Check baseTokenURI
b03321b
@xavikh
Test supportsInterface
e9a8d16
@xavikh
Allow resets outside voting period
2e9db89
@xavikh
Fix comment
997c9e7
@xavikh
Cleanup comment
61cbba8
@jordaniza
feat: migration functionality
7cf3d68
@jordaniza
update curve coeff and storage gap
585d9b8
@xavikh
Fix inheritance order
dd017da
@xavikh
Reduce Lock gap
b095eb0
@jordaniza
Merge pull request #40 from aragon/feat/migration
f5cbe5b 
feat: migration functionality
jordaniza and others added 23 commits February 14, 2025 20:18
@jordaniza
f: adds support for forge 1.0.0 in test cases
0698df9
@jordaniza
Merge branch 'f/foundry-v1' into f/rd-1193-foundry-upgrades-no-submod…
a2c3fff 
…ules
@jordaniza
poc for v1 testing
82e633b
@jordaniza
fixed imports for the fork and factory
28f76fa
@jordaniza
working now for v1_1_0
f24ff3f
@jordaniza
added the lock v1
4002987
@jordaniza
Moved factory tests to unit tests, adjusted makefile
4bd1b2a
@jordaniza
clean makefile and fix fork tests
b384de8
@jordaniza
update the test runner
addf9ee
@jordaniza
merged foundry upgrades
fd752d0
@jordaniza
added the upgrade tests for the lock
d456eea
@jordaniza
modified the test runner
4436aae
@jordaniza
missing merge conflict fixes
786ad89
@jordaniza
fixed broken import
ff31eb3
@jordaniza
merged the gh actions branch
0585ed8
@jordaniza
regression test
dba13eb
@jordaniza
Added safe upgrades to the regression test
2b5dc8e
@jordaniza
cleanup comments
6e126cf
@jordaniza
rename some files:
e33289a
@jordaniza
script file
f63b555
@jordaniza
removed the lock file from the repo
5c031e7
@jordaniza
added the ipfs file
fe47a38
@jordaniza
Merge branch 'deploy/mode' into deploy/mode-upgrade-v2
4f9b57a
xavikh
xavikh reviewed Feb 20, 2025
View reviewed changes
Copy link
Contributor

@xavikh xavikh left a comment

Choose a reason for hiding this comment

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

Looks good! Just some minor comments

xavikh
xavikh reviewed Feb 20, 2025
View reviewed changes
@jordaniza jordaniza merged commit 653ee26 into deploy/mode Feb 25, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xavikh xavikh xavikh left review comments

@novaknole novaknole novaknole left review comments

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

Successfully merging this pull request may close these issues.
4 participants
@jordaniza @xavikh @novaknole @brickpop