fix pyright for push.py (#479) #327
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: py | |
on: | |
pull_request: | |
push: | |
branches: [main] | |
jobs: | |
build: | |
runs-on: ubuntu-latest | |
strategy: | |
# As of 2024-11-08, this test takes roughly 60 seconds. There is not much benefit to fail-fast. | |
fail-fast: false | |
matrix: | |
python-version: | |
- "3.8" | |
- "3.9" | |
- "3.10" | |
- "3.11" | |
- "3.12" | |
- "3.13" | |
node_version: | |
- "20" | |
steps: | |
- uses: actions/checkout@v4 | |
- name: Set up Python ${{ matrix.python-version }} | |
uses: actions/setup-python@v5 | |
with: | |
python-version: ${{ matrix.python-version }} | |
- name: Set up Node.js ${{ matrix.node-version }} for Pyright | |
uses: actions/setup-node@v4 | |
with: | |
node-version: ${{ matrix.node-version }} | |
registry-url: "https://registry.npmjs.org" | |
- name: Set up pnpm | |
uses: pnpm/action-setup@v4 | |
with: | |
version: 8 | |
- name: Install dependencies | |
run: | | |
python -m pip install --upgrade pip setuptools build twine openai pylint | |
- name: Test whether the Python SDK can be installed | |
run: | | |
python -m pip install -e ./core/py[all] | |
python -m pip install -e ./py[all] | |
- name: Test whether the Python SDK can be imported | |
run: | | |
python -c 'import braintrust' | |
- name: Type check the Python SDK with Pyright | |
# TODO: Make Pyright errors fail the build. | |
run: | | |
pnpx pyright@1.1.388 --level error $(git ls-files '*.py') || true | |
- name: Lint the Python SDK | |
run: | | |
pylint --errors-only $(git ls-files '*.py') |