Skip to content

Commit

Permalink
fixes: readme and changelog
Browse files Browse the repository at this point in the history
  • Loading branch information
atkinsonholly authored and mvanmeerbeck committed Oct 9, 2023
1 parent e514e5d commit 57bb9d6
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 54 deletions.
7 changes: 6 additions & 1 deletion packages/asset/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,14 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [1.0.0] - 2023-10-06
## [Unreleased]

- first official version using release-it
- updates to readme
- fixes to changelog

## [1.0.0] - 2023-10-06

- as of this date we have deployed Catalyst, OperatorFilterSubscription, RoyaltyManager, RoyaltySplitter on Polygon-mainnet (note: Asset, AssetCreate, AssetReveal, AuthSuperValidator have not been deployed on Polygon-mainnet yet)
- move @openzeppelin contracts to dependencies (instead of dev deps)
- release-it and release-it hooks added
61 changes: 8 additions & 53 deletions packages/asset/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
# <PACKAGE>

The Sandbox Asset package for deploying on Polygon
The Sandbox Asset package for deploying on Polygon, consisting of the below contracts.

Asset (ERC1155) L2 token.
Asset's user-facing contracts: AssetCreate, AssetReveal.
AuthSuperValidator.
Catalyst (ERC1155) L2 token.

## Running the project locally

Expand All @@ -16,56 +21,6 @@ Formatting: Run `yarn prettier` to check and `yarn prettier:fix` to fix formatti

Linting: Run `yarn lint` to check and `yarn lint:fix` to fix static analysis errors

## Package structure and minimum standards

#### A NOTE ON DEPENDENCIES

1. Add whatever dependencies you like inside your package; this template is for hardhat usage. OpenZeppelin contracts
are highly recommended and should be installed as a dev dependency
2. For most Pull Requests there should be minimum changes to `yarn.lock` at root level
3. Changes to root-level dependencies are permissible, however they should not be downgraded
4. Take care to run `yarn` before pushing your changes
5. You shouldn't need to install dotenv since you won't be deploying inside this package (see below)

#### UNIT TESTING

1. Unit tests are to be added in `packages/<package>/test`
2. Coverage must meet minimum requirements for CI to pass
3. `getSigners` return an array of addresses, the first one is the default `deployer` for contracts, under no
circumstances should tests be written as `deployer`
4. It's permissible to create mock contracts at `packages/<package>/contracts/mock` e.g. for third-party contracts
5. Tests must not rely on any deploy scripts from the `deploy` package; your contracts must be deployed inside the test
fixture. See `test/fixtures.ts`

# Deployment

Each package must unit-test the contracts by running everything inside the `hardhat node`. Deployment to "real"
networks, configuration of our environment and integration tests must be done inside the `deploy` package.

The `deploy` package only imports `.sol` files. The idea is to recompile everything inside it and manage the entire
deploy strategy from one place.

1. Your deploy scripts should not be included inside `packages/<package>`: deploy scripts live inside `packages/deploy/`
2. The `deploy` package doesn't use the hardhat config file from the specific package. Instead, it
uses `packages/deploy/hardhat.config.ts`
3. You will need to review `packages/deploy/hardhat.config.ts` and update it as needed for any new namedAccounts you
added to your package
4. When it comes to deploy time, it is preferred to include deploy scripts and end-to-end tests as a separate PR
5. The named accounts inside the `deploy` package must use the "real-life" values
6. Refer to the readme at `packages/deploy` to learn more about importing your package

#### INTEGRATION TESTING

1. End-to-end tests live at `packages/deploy/`
2. You must add end-to-end tests ahead of deploying your package. Importantly, these tests should verify deployment and
initialization configuration

# A NOTE ON MAKING PULL REQUESTS

1. Follow the PR template checklist
2. Your PR will not be approved if the above criteria are not met




## Deployment

This package exports the contract source code, for deployments see: [@sandbox-smart-contract/deploy](../deploy) package.

0 comments on commit 57bb9d6

Please sign in to comment.