Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Contracts management #391

Open
wants to merge 58 commits into
base: main
Choose a base branch
from
Open

Contracts management #391

wants to merge 58 commits into from

Conversation

imyourm8
Copy link
Contributor

The purpose of this PR is to improve the way our smart contracts are deployed, updated, and verified to improve the overall quality and quality of life.

List of major changes:

  • Drop truffle as a go-to framework and embrace Hardhat.
  • Typescript.
  • Single command deployment to root and child chains.
  • Release version tracking to keep track of addresses and storage layouts.
  • Verify contracts bytecode on-chain and storage layout violations.
  • Simple conditional compilation for solidity. Code generation is based on the selected network.
  • Scripts to upgrade contracts easily.

zorancuc and others added 30 commits August 27, 2021 10:28
I have disabled tests for contracts we don't use in production. Such as all predicates except Burn Only. Draining of the stake manager too, it uses Gnosis Safe there, which suddenly fails now, due to broken signing.
Update current release file and include storage layouts.
Used previously in truffle migrations
@imyourm8 imyourm8 requested a review from jdkanani December 1, 2021 08:58
@@ -1,6 +1,6 @@
pragma solidity ^0.5.2;

import {RootChainStorage} from "./RootChainStorage.sol";
import {RootChainStorage} from "./RootChainStorage.sol";

Choose a reason for hiding this comment

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

added trailing whitespace?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants