Skip to content

Commit

Permalink
chore: Bump isort and flake
Browse files Browse the repository at this point in the history
  • Loading branch information
samwerner committed Feb 24, 2021
1 parent c1ee0d1 commit 0cb8789
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 24 deletions.
11 changes: 6 additions & 5 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@ repos:
rev: 19.3b0
hooks:
- id: black
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v2.4.0

- repo: https://gitlab.com/pycqa/flake8
rev: 3.8.4
hooks:
- id: flake8
- repo: https://github.com/pre-commit/mirrors-isort
rev: v4.3.21
- repo: https://github.com/PyCQA/isort
rev: 5.7.0
hooks:
- id: isort

default_language_version:
python: python3.8
python: python3.8
28 changes: 14 additions & 14 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@ Curve allows users to trade between correlated cryptocurrencies with a bespoke l

### Dependencies

* [python3](https://www.python.org/downloads/release/python-368/) from version 3.6 to 3.8, python3-dev
* [brownie](https://github.com/iamdefinitelyahuman/brownie) - tested with version [1.12.0](https://github.com/eth-brownie/brownie/releases/tag/v1.12.0)
* [ganache-cli](https://github.com/trufflesuite/ganache-cli) - tested with version [6.11.0](https://github.com/trufflesuite/ganache-cli/releases/tag/v6.11.0)
* [brownie-token-tester](https://github.com/iamdefinitelyahuman/brownie-token-tester) - tested with version [0.0.3](https://github.com/iamdefinitelyahuman/brownie-token-tester/releases/tag/v0.0.3)
- [python3](https://www.python.org/downloads/release/python-368/) from version 3.6 to 3.8, python3-dev
- [brownie](https://github.com/iamdefinitelyahuman/brownie) - tested with version [1.12.0](https://github.com/eth-brownie/brownie/releases/tag/v1.12.0)
- [ganache-cli](https://github.com/trufflesuite/ganache-cli) - tested with version [6.11.0](https://github.com/trufflesuite/ganache-cli/releases/tag/v6.11.0)
- [brownie-token-tester](https://github.com/iamdefinitelyahuman/brownie-token-tester) - tested with version [0.1.0](https://github.com/iamdefinitelyahuman/brownie-token-tester/releases/tag/v0.1.0)

Curve contracts are compiled using [Vyper](https://github.com/vyperlang/vyper), however installation of the required Vyper versions is handled by Brownie.

Expand All @@ -31,8 +31,8 @@ pip install -r requirements.txt

### Organization and Workflow

* New Curve pools are built from the contract templates at [`contracts/pool-templates`](contracts/pool-templates)
* Once deployed, the contracts for a pool are added to [`contracts/pools`](contracts/pools)
- New Curve pools are built from the contract templates at [`contracts/pool-templates`](contracts/pool-templates)
- Once deployed, the contracts for a pool are added to [`contracts/pools`](contracts/pools)

See the documentation within [`contracts`](contracts) and it's subdirectories for more detailed information on how to get started developing on Curve.

Expand Down Expand Up @@ -62,19 +62,19 @@ To deploy a new pool:
2. Edit the configuration settings within [`scripts/deploy.py`](scripts/deploy.py).
3. Test the deployment locally against a forked mainnet.

```bash
brownie run deploy --network mainnet-fork -I
```
```bash
brownie run deploy --network mainnet-fork -I
```

When the script completes it will open a console. You should call the various getter methods on the deployed contracts to ensure the pool has been configured correctly.
When the script completes it will open a console. You should call the various getter methods on the deployed contracts to ensure the pool has been configured correctly.

4. Deploy the pool to the mainnet.

```bash
brownie run deploy --network mainnet
```
```bash
brownie run deploy --network mainnet
```

Be sure to open a pull request that adds the deployment addresses to the pool `README.md`.
Be sure to open a pull request that adds the deployment addresses to the pool `README.md`.

## Audits and Security

Expand Down
4 changes: 2 additions & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
black==19.10b0
eth-brownie>=1.13.0,<2.0.0
flake8==3.7.9
isort==4.3.21
flake8==3.8.4
isort==5.7.0
brownie-token-tester>=0.1.0
1 change: 0 additions & 1 deletion tests/pools/aave/integration/test_curve_aave.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import pytest
from brownie.test import given, strategy
from hypothesis import settings

from simulation import Curve

pytestmark = pytest.mark.skip_meta
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import pytest
from brownie.test import given, strategy
from hypothesis import settings

from simulation import Curve

# do not run this test on pools without lending or meta pools
Expand Down
1 change: 0 additions & 1 deletion tests/pools/common/integration/test_simulate_exchange.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import pytest
from brownie.test import given, strategy
from hypothesis import settings

from simulation import Curve

# do not run this test on pools without lending or meta pools
Expand Down

0 comments on commit 0cb8789

Please sign in to comment.