Disabled publishing of contract artifacts and maintainers container to keep-dev #318
Conversation
It's a temporary change for the purpose of having a stable demo environment. We'll bring it back later when we figure out how to version-tag our deployments properly.
| context: keep-dev | ||
| requires: | ||
| - migrate_contracts | ||
| # - build_initcontainer: |
There was a problem hiding this comment.
I'm sure you did this to save time, to be clear it's not strictly required....makes sense from a time saving perspective while we're not publishing though.
There was a problem hiding this comment.
Yep, that was my reasoning sir.
|
Mmmm do we need the |
You are right, though... in the Uniswap deployment PR we discovered some problems on CI during contract deployment after we merged that PR. I think it makes sense to leave this job running just to make sure we don't break something and discover it one month later. |
It's a temporary change for the purpose of having a stable demo environment. We'll bring it back later when we figure out how to version-tag our deployments properly.