Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] committed Nov 28, 2024
1 parent e0b51ac commit e1ca653
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 25 deletions.
18 changes: 9 additions & 9 deletions .github/workflows/e2e_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,14 @@ jobs:
runs-on: ubuntu-latest

steps:
# Checkout the repository
- name: Checkout Code
uses: actions/checkout@v4
# Checkout the repository
- name: Checkout Code
uses: actions/checkout@v4

# Set up Docker
- name: Set up Docker
uses: docker/setup-buildx-action@v2
# Set up Docker
- name: Set up Docker
uses: docker/setup-buildx-action@v2

# Run the Python script that handles the build and execution
- name: Run Python Script
run: python3 main.py local && python3 main.py test
# Run the Python script that handles the build and execution
- name: Run Python Script
run: python3 main.py local && python3 main.py test
23 changes: 11 additions & 12 deletions .github/workflows/pyright.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,18 @@ jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
cache: 'pip'
- uses: actions/checkout@v3
- uses: actions/setup-python@v4
with:
cache: "pip"

- run: |
python -m venv .venv
source .venv/bin/activate
pip install -r Docker/user_code_requirements.txt
- run: |
python -m venv .venv
source .venv/bin/activate
pip install -r Docker/user_code_requirements.txt
- run: echo "$PWD/.venv/bin" >> $GITHUB_PATH
- run: echo "$PWD/.venv/bin" >> $GITHUB_PATH

- uses: jakebailey/pyright-action@v2
with:
- uses: jakebailey/pyright-action@v2
with:
pylance-version: latest-release

6 changes: 3 additions & 3 deletions Docker/docker-compose-user-code.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ services:
DAGSTER_CURRENT_IMAGE: "dagster_user_code_image"
volumes:
- ../userCode:/opt/dagster/app/userCode
# When materialized via the UI, dagster runs the
# When materialized via the UI, dagster runs the
# user code inside the webserver container
# However, if we are just running pytest we need to have direct
# access to the docker sock inside the user code container
- /var/run/docker.sock:/var/run/docker.sock
- /var/run/docker.sock:/var/run/docker.sock

networks:
- dagster_network
env_file: "../.env"
Expand Down
1 change: 0 additions & 1 deletion userCode/test/test_e2e.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,3 @@ def test_materialize_configs():
selection=["nabu_config", "gleaner_config", "docker_client_environment"],
)
assert result.success

0 comments on commit e1ca653

Please sign in to comment.