feat: improved update checklist #396
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The checkslist allows version specific tasks in a append only format
brickpop
previously approved these changes
Jun 2, 2023
heueristik
force-pushed
the
feature/improve-deployment-checklist
branch
from
f4d7670
to
7c790ce
Compare
brickpop
approved these changes
Jun 2, 2023
Rekard0
approved these changes
Jun 2, 2023
mathewmeconry
added a commit
that referenced
this pull request
Jun 22, 2023
* OS-459 : Subgraph update to v1.3.0 (#393) * rename 120 to 130 * fix dates * update change log * revert contracts test (renaming mistake) * remove unnecessary log * feat: improved update checklist (#396) * feat: improved update checklist * feat: update PR checklist * feat: improved the update checklist * fix: remove checklist item * feat: added publishing flow by Jordi * chore: Updates update_checklist to reflect multiple version The checkslist allows version specific tasks in a append only format * feat: formatting * fix: typo * feat: restructure README section about publication * revert: re-formatting * fix: wrong link --------- Co-authored-by: Mathias Scherer <mathias@aragon.org> * Feat/update scrips (#394) * feat: updates networks to use v1.3.0 update scripts fix: changes mumbai rpc url to use the same infura endpoint as the others * feat(OS-429): replaces old ipfs endpoint with new one switches ipfs-0.aragon.network with the load balances prod.ipfs.aragon.network endpoint * fix(OS-429): replaces old api keys with valid ones * fix: reverts active_contracts.json to old state the values in active_contracts.json for mumbai were already the non applied update addresses * feat: adds node version requirement to checklists * feat: adds new functions to deploy helpers fix: uses correct addresses during DAOFactory v1.3.0 update * fix: add initializeFrom call to the managing DAO proposal (#395) * fix: add initializeFrom call to the managing DAO proposal * fix: empty data --------- Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> * Clarifying NatSpec comment in the `DAORegistry.register` function (#403) * feat: added clarifying NatSpec comment * feat: improved docs * Tiny typo fix --------- Co-authored-by: Jør∂¡ <itsbrickpop@gmail.com> * feat: improved NatSpec comment (#405) * feat: added revert case to applySingleTargetPermissions (#400) * Dops 531 update check user perm gh actions osx (#407) * fix(ci): new gh actions in comment-triggers to check user permission * Added revert case to grantWithCondition (#401) * feat: added revert case to grantWithCondition * fix: removed empty else branch * fix: NatSpec corrections * fix: error usage and tests * fix: test description * feat: optimization * fix: corrected build metadata (#411) * fix: corrected build metadata * fix: added missing item to the change list * feat: updated deployment checklist * feat: improved checklist item * bump package version (#413) * bump package version * add satsuma ipfs gateway * fix polygon manifest (#414) * Added ProtocolVersion to PluginRepoFactory and PluginRepo (#412) * feat: added ProtocolVersion to PluginRepoFactory and PluginRepo * fix: remove wrong ERC-165 usage * chore: maintained changelog * add an extra check (#410) * ERC-165 support for PermissionConditions (#402) * feat: added revert case to grantWithCondition * fix: removed empty else branch * fix: NatSpec corrections * fix: error usage and tests * fix: test description * feat: optimization * feat: added PermissionConditionBase and ERC-165 support * feat: improved error handling and additional test * feat: improved changelog * feat: use address instead of explicit type * fix: tests * feat: improved error handling * feat: added PermissionConditionUpgradeable base contract * fix: forgotten rename * fix: forgotten rename * fix: contract name and NatSpec * fix: typo * Update protocol upgrade deploy scripts (#409) * feat: added deploy scripts and correction in existing scripts * feat: maintained and improved update checklist * fix: remove accidentally duplicated files * revert: renaming * feat: improved managing DAO txns descriptions * feat: improve logs * feat: move import * fix: wrong checklist items * fix: no plural * feat: refactored tests and added contracts * feat: improve update test * Fix: Add decimals to ERC20WrapperContract (#416) * add decimals to erc20wrappercontract * update package version * fix tests * Documentation update: How To Guides overview (#367) * updating documentation to be more tutorial like and use case based * documentation update * added fixes to docs based on code reviews * formatting * fixed review from michael and jordim * Update packages/contracts/docs/osx/02-how-to-guides/index.md * removed blank linke; * fixing titles and revising comments on PR * fixed michaels review * add documentation on non-upgradeable plugin to include a get started guide * adding the format check * fix jordis comments on technally incorrect info and typos * fixing plugin setup corrections * fixed comments based on jordis input - particularly regarding the setup processing and the relationship between the plugin and the dao * format text * fixed jordi comments * removing storage gap * fixed title * OS-516 : add missing artifacts (#408) * add missing artifacts * Apply suggestions from code review Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> --------- Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> * Fix CI/CD by supporting tags (#417) * fix: use explicit working directory paths * Revert "fix: use explicit working directory paths" This reverts commit 09ef365. * feat: support tags * fix: rename variable --------- Co-authored-by: Rekard0 <5880388+Rekard0@users.noreply.github.com> Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> Co-authored-by: Jør∂¡ <itsbrickpop@gmail.com> Co-authored-by: Roger Carhuatocto <chilcano@intix.info> Co-authored-by: josemarinas <36479864+josemarinas@users.noreply.github.com> Co-authored-by: juliettech <juliettech.1393@gmail.com>
mathewmeconry
added a commit
that referenced
this pull request
Jul 11, 2023
* dev2main (#397) * OS-459 : Subgraph update to v1.3.0 (#393) * rename 120 to 130 * fix dates * update change log * revert contracts test (renaming mistake) * remove unnecessary log * feat: improved update checklist (#396) * feat: improved update checklist * feat: update PR checklist * feat: improved the update checklist * fix: remove checklist item * feat: added publishing flow by Jordi * chore: Updates update_checklist to reflect multiple version The checkslist allows version specific tasks in a append only format * feat: formatting * fix: typo * feat: restructure README section about publication * revert: re-formatting * fix: wrong link --------- Co-authored-by: Mathias Scherer <mathias@aragon.org> * Feat/update scrips (#394) * feat: updates networks to use v1.3.0 update scripts fix: changes mumbai rpc url to use the same infura endpoint as the others * feat(OS-429): replaces old ipfs endpoint with new one switches ipfs-0.aragon.network with the load balances prod.ipfs.aragon.network endpoint * fix(OS-429): replaces old api keys with valid ones * fix: reverts active_contracts.json to old state the values in active_contracts.json for mumbai were already the non applied update addresses * feat: adds node version requirement to checklists * feat: adds new functions to deploy helpers fix: uses correct addresses during DAOFactory v1.3.0 update * fix: add initializeFrom call to the managing DAO proposal (#395) * fix: add initializeFrom call to the managing DAO proposal * fix: empty data --------- Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> * Clarifying NatSpec comment in the `DAORegistry.register` function (#403) * feat: added clarifying NatSpec comment * feat: improved docs * Tiny typo fix --------- Co-authored-by: Jør∂¡ <itsbrickpop@gmail.com> * feat: improved NatSpec comment (#405) * feat: added revert case to applySingleTargetPermissions (#400) * Dops 531 update check user perm gh actions osx (#407) * fix(ci): new gh actions in comment-triggers to check user permission * Added revert case to grantWithCondition (#401) * feat: added revert case to grantWithCondition * fix: removed empty else branch * fix: NatSpec corrections * fix: error usage and tests * fix: test description * feat: optimization * fix: corrected build metadata (#411) * fix: corrected build metadata * fix: added missing item to the change list * feat: updated deployment checklist * feat: improved checklist item * bump package version (#413) * bump package version * add satsuma ipfs gateway * fix polygon manifest (#414) * Added ProtocolVersion to PluginRepoFactory and PluginRepo (#412) * feat: added ProtocolVersion to PluginRepoFactory and PluginRepo * fix: remove wrong ERC-165 usage * chore: maintained changelog * add an extra check (#410) * ERC-165 support for PermissionConditions (#402) * feat: added revert case to grantWithCondition * fix: removed empty else branch * fix: NatSpec corrections * fix: error usage and tests * fix: test description * feat: optimization * feat: added PermissionConditionBase and ERC-165 support * feat: improved error handling and additional test * feat: improved changelog * feat: use address instead of explicit type * fix: tests * feat: improved error handling * feat: added PermissionConditionUpgradeable base contract * fix: forgotten rename * fix: forgotten rename * fix: contract name and NatSpec * fix: typo * Update protocol upgrade deploy scripts (#409) * feat: added deploy scripts and correction in existing scripts * feat: maintained and improved update checklist * fix: remove accidentally duplicated files * revert: renaming * feat: improved managing DAO txns descriptions * feat: improve logs * feat: move import * fix: wrong checklist items * fix: no plural * feat: refactored tests and added contracts * feat: improve update test * Fix: Add decimals to ERC20WrapperContract (#416) * add decimals to erc20wrappercontract * update package version * fix tests * Documentation update: How To Guides overview (#367) * updating documentation to be more tutorial like and use case based * documentation update * added fixes to docs based on code reviews * formatting * fixed review from michael and jordim * Update packages/contracts/docs/osx/02-how-to-guides/index.md * removed blank linke; * fixing titles and revising comments on PR * fixed michaels review * add documentation on non-upgradeable plugin to include a get started guide * adding the format check * fix jordis comments on technally incorrect info and typos * fixing plugin setup corrections * fixed comments based on jordis input - particularly regarding the setup processing and the relationship between the plugin and the dao * format text * fixed jordi comments * removing storage gap * fixed title * OS-516 : add missing artifacts (#408) * add missing artifacts * Apply suggestions from code review Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> --------- Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> * Fix CI/CD by supporting tags (#417) * fix: use explicit working directory paths * Revert "fix: use explicit working directory paths" This reverts commit 09ef365. * feat: support tags * fix: rename variable --------- Co-authored-by: Rekard0 <5880388+Rekard0@users.noreply.github.com> Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> Co-authored-by: Jør∂¡ <itsbrickpop@gmail.com> Co-authored-by: Roger Carhuatocto <chilcano@intix.info> Co-authored-by: josemarinas <36479864+josemarinas@users.noreply.github.com> Co-authored-by: juliettech <juliettech.1393@gmail.com> * R/1.3.0 rc0 (#429) * chore: updates contracts changelog with all the changes * chore: fixes typos in UPDATE_CHECKLIST.md * feat(rc0): managingDAO deploy proposal is now json * fix(rc0): error in update checklist copying wrong env variables * Provide Governance Token Bases to `TokenVotingSetup` Constructor as Arguments to Avoid `initCode` size limit on Goerli (#425) * fix: provide governance token bases as constructor arguments * feat: verify governance token bases * fix: adapt script for fresh deployment * fix: typo * fix: includes plugin bases in contract verification * chore(1.3.0): update active_contract.json with new addresses of testnets chore(1.3.0): update Releases.md with new releases in testnets * chore: update npm package versions to 1.3.0-rc0 --------- Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> * chore: remove old managingDAO address of goerli and mumbai chore: update Changelog to reflect v1.3.0-rc0 release --------- Co-authored-by: Rekard0 <5880388+Rekard0@users.noreply.github.com> Co-authored-by: Michael Heuer <20623991+Michael-A-Heuer@users.noreply.github.com> Co-authored-by: Jør∂¡ <itsbrickpop@gmail.com> Co-authored-by: Roger Carhuatocto <chilcano@intix.info> Co-authored-by: josemarinas <36479864+josemarinas@users.noreply.github.com> Co-authored-by: juliettech <juliettech.1393@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Task: None
Type of change