Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci/python] Don't build wheels on each PR push #697

Merged
merged 5 commits into from
Jan 11, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ jobs:
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true

# TODO: merge with python-packaging.yml -- https://github.com/single-cell-data/TileDB-SOMA/issues/700
- name: Build wheel distribution
run: python -m pip -v wheel --no-deps --wheel-dir=dist-wheel apis/python
env:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,3 +27,8 @@ jobs:
# python -m pip install mypy types-setuptools typeguard
python -m pip install mypy==0.982 types-setuptools 'attrs>=22.1'
python -m mypy .

- name: Check C++ Format
run: |
./scripts/run-clang-format.sh . clang-format 0 \
$(find libtiledbsoma -name "*.cc" -or -name "*.h")
Original file line number Diff line number Diff line change
@@ -1,14 +1,13 @@
name: TileDB-SOMA python sdist & wheels

on:
# Normally the smoke test only runs on merge to main. To run it on a PR CI, comment out
# `workflow_dispatch:` and uncomment `pull_request:`.
# I.e. on request
workflow_dispatch:
# Normally the smoke test only runs on merge to main. To run it on a PR CI, uncomment `pull_request:`
# pull_request:
push:
branches: [main]
paths-ignore:
- 'apis/r/**'
# branches: [main]
# paths-ignore:
# - 'apis/r/**'
release:
types: [published]

Expand All @@ -30,6 +29,7 @@ jobs:
name: sdist
path: apis/python/dist/

# TODO: merge with python-packaging.yml -- https://github.com/single-cell-data/TileDB-SOMA/issues/700
wheels:
# Note: tries all supported Python versions as specified in apis/python/setup.py
name: Build wheels on ${{ matrix.os }}
Expand Down
2 changes: 1 addition & 1 deletion apis/python/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Overview

This is a POC Python implementation of the proposed [Unified Single-cell Data Model](https://github.com/single-cell-data/SOMA).
This is a Python implementation of the [Unified Single-cell Data Model](https://github.com/single-cell-data/SOMA).

This branch, `main`, implements the [updated specfication](https://github.com/single-cell-data/SOMA/blob/main/abstract_specification.md). Please also see the `main-old` branch which implements the [original specification](https://github.com/single-cell-data/TileDB-SOMA/blob/main-old/spec/specification.md).

Expand Down