Skip to content

Commit

Permalink
Build fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
DanielSchiavini committed Nov 22, 2023
1 parent 4cc75a1 commit e9eaabe
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 9 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/.pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@ jobs:
- name: Checkout
uses: actions/checkout@v4

- name: Setup Python 3.10.13
- name: Setup Python
uses: actions/setup-python@v4
with:
python-version: 3.10.13
python-version: 3.11

- name: Pre-commit checks
uses: pre-commit/action@v3.0.0
2 changes: 1 addition & 1 deletion .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@ jobs:
- name: Run Tests
run: pytest
env:
DRPC_KEY: ${{ secrets.DRPC_KEY }}
RPC_ETHEREUM: ${{ secrets.RPC_ETHEREUM }}
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@ RUN pip install --upgrade pip && \
pip install --no-cache-dir -r requirements.txt

FROM base as pre-commit
RUN apt-get update && apt-get install git
RUN apt-get update && apt-get install -y git
COPY . .
CMD pre-commit run --all-files
CMD git add . && pre-commit run --all-files

FROM base as test
COPY . .
Expand Down
1 change: 0 additions & 1 deletion docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,5 @@ services:
context: ./
target: pre-commit
container_name: metaregistry-pre-commit
command: pre-commit run --all-files
volumes:
- ./:/usr/app
4 changes: 1 addition & 3 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,7 @@ def pytest_sessionstart():
global ALL_POOLS

# connect to the network. TODO: use Drpc-Key header instead of GET param
boa.env.fork(
f"https://lb.drpc.org/ogrpc?network=ethereum&dkey={environ['DRPC_KEY']}"
)
boa.env.fork(url=environ["RPC_ETHEREUM"])

# store instance of registries globally, so we don't have to recreate multiple times when generating tests.
# TODO: Can we move these to fixtures?
Expand Down

0 comments on commit e9eaabe

Please sign in to comment.