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

fix: Add a database to the Deploy Campaign #58

Merged
merged 60 commits into from
Nov 13, 2023
Merged

fix: Add a database to the Deploy Campaign #58

merged 60 commits into from
Nov 13, 2023

Conversation

Whytecrowe
Copy link
Collaborator

No description provided.

@Whytecrowe Whytecrowe changed the base branch from development to deploy-proto October 13, 2023 18:57
@codecov
Copy link

codecov bot commented Oct 13, 2023 •
edited
Loading

Codecov Report

Merging #58 (fa550c1) into feat/deploy-campaign (dd1a9ad) will not change coverage.
Report is 1 commits behind head on feat/deploy-campaign.
The diff coverage is n/a.

❗ Current head fa550c1 differs from pull request most recent head 614f1f4. Consider uploading reports for the commit 614f1f4 to get more accurate results

Additional details and impacted files 
@@                  Coverage Diff                  @@
##           feat/deploy-campaign      #58   +/-   ##
=====================================================
  Coverage                 99.80%   99.80%           
=====================================================
  Files                        11       11           
  Lines                       502      502           
  Branches                    124      124           
=====================================================
  Hits                        501      501           
  Misses                        1        1

@Whytecrowe
add some versioning logic and connect to mongo adapter in the RootReg…
09ace5e 
…istrar test
@Whytecrowe
add proper versioning logic for start and finish
208fcd5
@Whytecrowe
change docker compose port to 27018
97fbc4f
@Whytecrowe
add mongo test helper, update the campaign launcher and registrar test
990132e
@Whytecrowe
rename mongo connector to mongo adapter, add module level var to use …
d3194a1 
…the same adapter for all tests and a getter function
@Whytecrowe
refine test mongo functions and add a stop function
3f0a2e8
@Whytecrowe
modify some types and base deploy mission to not save deploy args
27a5807
@Whytecrowe
rework logger functions to save to a module var so that we use the sa…
cfa3dd9 
…me logger everywhere
@Whytecrowe
add global setup methods for mocha tests, override hardhat test task …
90075db 
…to include the setup and teardown, add script to stop the docker container, but don't use it yet.
@Whytecrowe
modify runZnsCampaign() and RootRegistrar test for the new setup
6a75bb4
@Whytecrowe
fix some naming
6be819f
@Whytecrowe
Merge branch 'deploy-proto' into deploy/add-database
2b5d0fc
@Whytecrowe
remove redundant prop from campaign config
af19c65
@Whytecrowe
remove duplicated imports it test
94162a5
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Nov 7, 2023 •
edited
Loading

fix: Add a database to the Deploy Campaign

Generated at commit: 2fce3b69311b27a27e8d25c8a0537d1cfd9a77dc

🚨 Vulnerabilities Summary

Process Issues Results
Contract Inspector note
low
Total		 21
5
26
Dependency Checker Total 0

For more details view the full report in OpenZeppelin Code

@Whytecrowe
try removing remote docker
8407c06
@Whytecrowe
Revert "try removing remote docker"
3bb1d91 
This reverts commit 8407c06.
@Whytecrowe
try changing port
43a0cae
@Whytecrowe
try with different uri
15d2377
@Whytecrowe
try env var
2de738e
@Whytecrowe
revert
2277b0e
@Whytecrowe
fix errors in base deploy mission and move zns names to it's own file
072e646
@Whytecrowe
make mongo stop and async call, refactor dropDB to make it work even …
9c39a62 
…if container is stopped
@Whytecrowe
add comments
3949bd9
@Whytecrowe
lock specific version of zToken package
f96e937
@Whytecrowe
extend hardhat deployer to be able to get bytecode and factory
4d3b68e
@Whytecrowe
add hardcoded mainnet data for meow to the repo
95b4313
@Whytecrowe
extend meow naming and deploy logic to support both mocked and mainne…
26418d9 
…t contract
@Whytecrowe
rename and move token class
5cf4981
@Whytecrowe
fix Circle config to properly work with mongo
3704772
@Whytecrowe
update solhint
718621f
@Whytecrowe
update yarn.lock
4c7d3c5
@Whytecrowe
change instances in campaign state from array to object
3b0ac41
@Whytecrowe
add and comment out the bytecode comparison tool
f65f718
@Whytecrowe
add db drop to the mocha teardown
e561b30
@Whytecrowe
rework campaign smoke test to start testing campaign
6964a49
@Whytecrowe
remove all redundant code in TreasuryDM
2fce3b6
@Whytecrowe
return proper exit code after tests
d169cca
@Whytecrowe
remove postDeploy hook from Treasury
037bb9e
@Whytecrowe
fix overriden test task to return failures instead of exiting
fa550c1
@Whytecrowe Whytecrowe changed the base branch from deploy-proto to feat/deploy-campaign November 13, 2023 22:36
@Whytecrowe Whytecrowe merged commit fb38a1a into feat/deploy-campaign Nov 13, 2023
@zero-devops
Copy link
Collaborator

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Whytecrowe @zero-devops