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

Commit

Permalink
merge master
Browse files Browse the repository at this point in the history
  • Loading branch information
danielgafni committed Nov 1, 2023
2 parents b8db2c2 + c7d912c commit 60c6828
Show file tree
Hide file tree
Showing 19 changed files with 1,348 additions and 508 deletions.
77 changes: 0 additions & 77 deletions .github/workflows/check.yml

This file was deleted.

135 changes: 135 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,135 @@
name: CI

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true

on:
workflow_dispatch:
push:
release:
types:
- created
# Sequence of patterns matched against refs/tags
tags:
- 'v*' # Push events to matching v*, i.e. v1.0, v20.15.10

jobs:
test:
name: test polars=${{ matrix.polars_version }} dagster=${{ matrix.dagster_version }} py=${{ matrix.py }} ${{ matrix.os }}
runs-on: ${{ matrix.os }}-latest
strategy:
fail-fast: false
matrix:
os:
- Ubuntu
# MacOs is commented out because: 1. For some reason it's very slow. 2. It never had OS-specific issues
# - MacOs
py:
- "3.11"
- "3.10"
- "3.9"
- "3.8"
dagster_version:
- "1.4.0"
# - "1.5.0" disabled until https://github.com/dagster-io/dagster/issues/17436 is fixed
polars_version:
- "0.17.0"
- "0.18.0"
- "0.19.0"
steps:
- name: Setup python for test ${{ matrix.py }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.py }}
- uses: actions/checkout@v2
- name: Install and configure Poetry
uses: snok/install-poetry@v1
with:
version: 1.4.2
virtualenvs-create: false
virtualenvs-in-project: false
installer-parallel: true
- name: Install dependencies
run: poetry install --all-extras --sync && pip install --force-reinstall polars~=${{ matrix.polars_version }} dagster~=${{ matrix.dagster_version }}
- name: Print polars info
run: python -c 'import polars; print(polars.show_versions())'
- name: Print dagster info
run: python -c 'import dagster; print(dagster.__version__)'
- name: Run tests
run: pytest -v .

lint:
name: lint polars=${{ matrix.polars_version }} dagster=${{ matrix.dagster_version }} py=${{ matrix.py }} ${{ matrix.os }}
runs-on: ${{ matrix.os }}-latest
strategy:
fail-fast: false
matrix:
os:
- Ubuntu
py:
- "3.11"
- "3.10"
- "3.9"
- "3.8"
dagster_version:
- "1.4.0"
# - "1.5.0" disabled until https://github.com/dagster-io/dagster/issues/17436 is fixed
polars_version:
- "0.17.0"
- "0.18.0"
- "0.19.0"
steps:
- name: Setup python for test ${{ matrix.py }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.py }}
- uses: actions/checkout@v2
- name: Install and configure Poetry
uses: snok/install-poetry@v1
with:
version: 1.4.2
virtualenvs-create: false
virtualenvs-in-project: false
installer-parallel: true
- name: Install dependencies
run: poetry install --all-extras --sync && pip install --force-reinstall polars~=${{ matrix.polars_version }} dagster~=${{ matrix.dagster_version }}
- name: Run pre-commit hooks
run: pre-commit run --all-files

qa_success:
name: QA Success
needs: [test, lint]
runs-on: ubuntu-latest
steps:
- name: Success
run: echo "QA Succeeded!"

publish:
needs: qa_success
if: contains(github.ref, 'refs/tags/')
name: Publish to PyPI
strategy:
fail-fast: false
runs-on: ubuntu-latest
steps:
- name: Setup Python
uses: actions/setup-python@v2
with:
python-version: 3.9
- uses: actions/checkout@v2
- name: Install and configure Poetry
uses: snok/install-poetry@v1
with:
version: 1.4.2
virtualenvs-create: false
virtualenvs-in-project: false
installer-parallel: true
- name: Install poetry-dynamic-versioning
run: poetry self add [email protected]
- name: Publish to PyPI
env:
PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }}
run: |
poetry config pypi-token.pypi $PYPI_TOKEN
poetry publish --build
38 changes: 0 additions & 38 deletions .github/workflows/release.yml

This file was deleted.

7 changes: 6 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,15 @@ repos:
entry: black .
language: system
pass_filenames: false
- id: black-docs
name: black-docs
entry: blacken-docs
language: system
pass_filenames: true
files: '\.md$'

- id: pyright
name: pyright
entry: pyright .
language: system
pass_filenames: false

Loading

0 comments on commit 60c6828

Please sign in to comment.