Copilot commented on code in PR #1375:
URL:
https://github.com/apache/datafusion-python/pull/1375#discussion_r2784571156
##########
.github/workflows/build.yml:
##########
@@ -15,61 +15,235 @@
# specific language governing permissions and limitations
# under the License.
-name: Python Release Build
+# Reusable workflow for running building
+# This ensures the same tests run for both debug (PRs) and release (main/tags)
builds
+
+name: Build
+
on:
- pull_request:
- branches: ["main"]
- push:
- tags: ["*-rc*"]
- branches: ["branch-*"]
+ workflow_call:
+ inputs:
+ build_mode:
+ description: 'Build mode: debug or release'
+ required: true
+ type: string
+ run_wheels:
+ description: 'Whether to build distribution wheels'
+ required: false
+ type: boolean
+ default: false
+
+env:
+ CARGO_TERM_COLOR: always
+ RUST_BACKTRACE: 1
jobs:
- build:
+ # ============================================
+ # Linting Jobs
+ # ============================================
+ lint-rust:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+ with:
+ toolchain: "nightly"
+ components: rustfmt
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+
+ - name: Check formatting
+ run: cargo +nightly fmt --all -- --check
+
+ lint-python:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
- name: Install Python
- uses: actions/setup-python@v6
+ uses: actions/setup-python@v5
with:
python-version: "3.12"
- - uses: astral-sh/setup-uv@v7
+ - uses: astral-sh/setup-uv@v6
with:
- enable-cache: true
+ enable-cache: true
- # Use the --no-install-package to only install the dependencies
- # but do not yet build the rust library
- name: Install dependencies
run: uv sync --dev --no-install-package datafusion
- # Update output format to enable automatic inline annotations.
- name: Run Ruff
run: |
uv run --no-project ruff check --output-format=github python/
uv run --no-project ruff format --check python/
- name: Run codespell
run: |
- uv run --no-project codespell --toml pyproject.toml
+ uv run --no-project codespell --toml pyproject.toml
+
+ lint-toml:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - name: Install taplo
+ uses: taiki-e/install-action@v2
+ with:
+ tool: taplo-cli
+
+ # if you encounter an error, try running 'taplo format' to fix the
formatting automatically.
+ - name: Check Cargo.toml formatting
+ run: taplo format --check
generate-license:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
- - uses: astral-sh/setup-uv@v7
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Install cargo-license
+ uses: taiki-e/install-action@v2
with:
- enable-cache: true
+ tool: cargo-license
- name: Generate license file
run: uv run --no-project python ./dev/create_license.py
+
- uses: actions/upload-artifact@v6
with:
name: python-wheel-license
path: LICENSE.txt
+ # ============================================
+ # Build - Linux x86_64
+ # ============================================
+ build-manylinux-x86_64:
+ needs: [generate-license, lint-rust, lint-python]
+ name: ManyLinux x86_64
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - run: rm LICENSE.txt
+ - name: Download LICENSE.txt
+ uses: actions/download-artifact@v7
+ with:
+ name: python-wheel-license
+ path: .
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Build (release mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'release'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --release --strip --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build (debug mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'debug'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build FFI test library
+ uses: PyO3/maturin-action@v1
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ working-directory: examples/datafusion-ffi-example
+ args: --out dist
+ rustup-components: rust-std
+
+ - run: cp examples/datafusion-ffi-example/dist/*.whl dist/
+
+ - name: Archive wheels
+ uses: actions/upload-artifact@v6
+ with:
+ name: dist-manylinux-x86_64
+ path: dist/*
+
+ # ============================================
+ # Build - Linux ARM64
+ # ============================================
+ build-manylinux-aarch64:
Review Comment:
`inputs.run_wheels` is passed by callers (false for PRs) but the
non-Linux-x86_64 wheel build jobs still execute in debug mode and only skip
uploading artifacts. This likely negates a chunk of the intended PR CI speedup.
Consider gating `build-manylinux-aarch64` and `build-python-mac-win` at the job
level with `if: inputs.run_wheels` (or `if: inputs.build_mode == 'release'`) so
PRs only build the single Linux x86_64 wheel needed for pytest.
```suggestion
build-manylinux-aarch64:
if: inputs.run_wheels
```
##########
.github/workflows/build.yml:
##########
@@ -120,13 +308,17 @@ jobs:
- name: Archive wheels
uses: actions/upload-artifact@v6
+ if: inputs.build_mode == 'release'
Review Comment:
`inputs.run_wheels` is passed by callers (false for PRs) but the
non-Linux-x86_64 wheel build jobs still execute in debug mode and only skip
uploading artifacts. This likely negates a chunk of the intended PR CI speedup.
Consider gating `build-manylinux-aarch64` and `build-python-mac-win` at the job
level with `if: inputs.run_wheels` (or `if: inputs.build_mode == 'release'`) so
PRs only build the single Linux x86_64 wheel needed for pytest.
##########
.github/workflows/build.yml:
##########
@@ -15,61 +15,235 @@
# specific language governing permissions and limitations
# under the License.
-name: Python Release Build
+# Reusable workflow for running building
+# This ensures the same tests run for both debug (PRs) and release (main/tags)
builds
+
+name: Build
+
on:
- pull_request:
- branches: ["main"]
- push:
- tags: ["*-rc*"]
- branches: ["branch-*"]
+ workflow_call:
+ inputs:
+ build_mode:
+ description: 'Build mode: debug or release'
+ required: true
+ type: string
+ run_wheels:
+ description: 'Whether to build distribution wheels'
+ required: false
+ type: boolean
+ default: false
+
+env:
+ CARGO_TERM_COLOR: always
+ RUST_BACKTRACE: 1
jobs:
- build:
+ # ============================================
+ # Linting Jobs
+ # ============================================
+ lint-rust:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+ with:
+ toolchain: "nightly"
+ components: rustfmt
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+
+ - name: Check formatting
+ run: cargo +nightly fmt --all -- --check
+
+ lint-python:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
- name: Install Python
- uses: actions/setup-python@v6
+ uses: actions/setup-python@v5
with:
python-version: "3.12"
- - uses: astral-sh/setup-uv@v7
+ - uses: astral-sh/setup-uv@v6
with:
- enable-cache: true
+ enable-cache: true
- # Use the --no-install-package to only install the dependencies
- # but do not yet build the rust library
- name: Install dependencies
run: uv sync --dev --no-install-package datafusion
- # Update output format to enable automatic inline annotations.
- name: Run Ruff
run: |
uv run --no-project ruff check --output-format=github python/
uv run --no-project ruff format --check python/
- name: Run codespell
run: |
- uv run --no-project codespell --toml pyproject.toml
+ uv run --no-project codespell --toml pyproject.toml
+
+ lint-toml:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - name: Install taplo
+ uses: taiki-e/install-action@v2
+ with:
+ tool: taplo-cli
+
+ # if you encounter an error, try running 'taplo format' to fix the
formatting automatically.
+ - name: Check Cargo.toml formatting
+ run: taplo format --check
generate-license:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
- - uses: astral-sh/setup-uv@v7
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Install cargo-license
+ uses: taiki-e/install-action@v2
with:
- enable-cache: true
+ tool: cargo-license
- name: Generate license file
run: uv run --no-project python ./dev/create_license.py
+
- uses: actions/upload-artifact@v6
with:
name: python-wheel-license
path: LICENSE.txt
+ # ============================================
+ # Build - Linux x86_64
+ # ============================================
+ build-manylinux-x86_64:
+ needs: [generate-license, lint-rust, lint-python]
+ name: ManyLinux x86_64
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - run: rm LICENSE.txt
+ - name: Download LICENSE.txt
+ uses: actions/download-artifact@v7
+ with:
+ name: python-wheel-license
+ path: .
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Build (release mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'release'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --release --strip --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build (debug mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'debug'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build FFI test library
+ uses: PyO3/maturin-action@v1
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ working-directory: examples/datafusion-ffi-example
+ args: --out dist
+ rustup-components: rust-std
+
+ - run: cp examples/datafusion-ffi-example/dist/*.whl dist/
+
+ - name: Archive wheels
+ uses: actions/upload-artifact@v6
+ with:
+ name: dist-manylinux-x86_64
+ path: dist/*
+
+ # ============================================
+ # Build - Linux ARM64
+ # ============================================
+ build-manylinux-aarch64:
+ needs: [generate-license, lint-rust, lint-python]
+ name: ManyLinux arm64
+ runs-on: ubuntu-24.04-arm
+ steps:
+ - uses: actions/checkout@v6
+
+ - run: rm LICENSE.txt
+ - name: Download LICENSE.txt
+ uses: actions/download-artifact@v7
+ with:
+ name: python-wheel-license
+ path: .
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Build (release mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'release'
+ with:
+ target: aarch64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --release --strip --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build (debug mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'debug'
+ with:
+ target: aarch64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Archive wheels
+ uses: actions/upload-artifact@v6
+ if: inputs.build_mode == 'release'
+ with:
+ name: dist-manylinux-aarch64
+ path: dist/*
+
+ # ============================================
+ # Build - macOS arm64 / Windows
+ # ============================================
build-python-mac-win:
- needs: [generate-license]
- name: Mac/Win
+ needs: [generate-license, lint-rust, lint-python]
+ name: macOS arm64 & Windows
Review Comment:
`inputs.run_wheels` is passed by callers (false for PRs) but the
non-Linux-x86_64 wheel build jobs still execute in debug mode and only skip
uploading artifacts. This likely negates a chunk of the intended PR CI speedup.
Consider gating `build-manylinux-aarch64` and `build-python-mac-win` at the job
level with `if: inputs.run_wheels` (or `if: inputs.build_mode == 'release'`) so
PRs only build the single Linux x86_64 wheel needed for pytest.
##########
.github/workflows/build.yml:
##########
@@ -15,61 +15,235 @@
# specific language governing permissions and limitations
# under the License.
-name: Python Release Build
+# Reusable workflow for running building
+# This ensures the same tests run for both debug (PRs) and release (main/tags)
builds
+
+name: Build
+
on:
- pull_request:
- branches: ["main"]
- push:
- tags: ["*-rc*"]
- branches: ["branch-*"]
+ workflow_call:
+ inputs:
+ build_mode:
+ description: 'Build mode: debug or release'
+ required: true
+ type: string
+ run_wheels:
+ description: 'Whether to build distribution wheels'
+ required: false
+ type: boolean
+ default: false
+
+env:
+ CARGO_TERM_COLOR: always
+ RUST_BACKTRACE: 1
jobs:
- build:
+ # ============================================
+ # Linting Jobs
+ # ============================================
+ lint-rust:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+ with:
+ toolchain: "nightly"
+ components: rustfmt
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+
+ - name: Check formatting
+ run: cargo +nightly fmt --all -- --check
+
+ lint-python:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
- name: Install Python
- uses: actions/setup-python@v6
+ uses: actions/setup-python@v5
with:
python-version: "3.12"
- - uses: astral-sh/setup-uv@v7
+ - uses: astral-sh/setup-uv@v6
with:
- enable-cache: true
+ enable-cache: true
- # Use the --no-install-package to only install the dependencies
- # but do not yet build the rust library
- name: Install dependencies
run: uv sync --dev --no-install-package datafusion
- # Update output format to enable automatic inline annotations.
- name: Run Ruff
run: |
uv run --no-project ruff check --output-format=github python/
uv run --no-project ruff format --check python/
- name: Run codespell
run: |
- uv run --no-project codespell --toml pyproject.toml
+ uv run --no-project codespell --toml pyproject.toml
+
+ lint-toml:
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - name: Install taplo
+ uses: taiki-e/install-action@v2
+ with:
+ tool: taplo-cli
+
+ # if you encounter an error, try running 'taplo format' to fix the
formatting automatically.
+ - name: Check Cargo.toml formatting
+ run: taplo format --check
generate-license:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v6
- - uses: astral-sh/setup-uv@v7
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Install cargo-license
+ uses: taiki-e/install-action@v2
with:
- enable-cache: true
+ tool: cargo-license
- name: Generate license file
run: uv run --no-project python ./dev/create_license.py
+
- uses: actions/upload-artifact@v6
with:
name: python-wheel-license
path: LICENSE.txt
+ # ============================================
+ # Build - Linux x86_64
+ # ============================================
+ build-manylinux-x86_64:
+ needs: [generate-license, lint-rust, lint-python]
+ name: ManyLinux x86_64
+ runs-on: ubuntu-latest
+ steps:
+ - uses: actions/checkout@v6
+
+ - run: rm LICENSE.txt
+ - name: Download LICENSE.txt
+ uses: actions/download-artifact@v7
+ with:
+ name: python-wheel-license
+ path: .
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Build (release mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'release'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --release --strip --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build (debug mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'debug'
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build FFI test library
+ uses: PyO3/maturin-action@v1
+ with:
+ target: x86_64-unknown-linux-gnu
+ manylinux: "2_28"
+ working-directory: examples/datafusion-ffi-example
+ args: --out dist
+ rustup-components: rust-std
+
+ - run: cp examples/datafusion-ffi-example/dist/*.whl dist/
+
+ - name: Archive wheels
+ uses: actions/upload-artifact@v6
+ with:
+ name: dist-manylinux-x86_64
+ path: dist/*
+
+ # ============================================
+ # Build - Linux ARM64
+ # ============================================
+ build-manylinux-aarch64:
+ needs: [generate-license, lint-rust, lint-python]
+ name: ManyLinux arm64
+ runs-on: ubuntu-24.04-arm
+ steps:
+ - uses: actions/checkout@v6
+
+ - run: rm LICENSE.txt
+ - name: Download LICENSE.txt
+ uses: actions/download-artifact@v7
+ with:
+ name: python-wheel-license
+ path: .
+
+ - name: Setup Rust
+ uses: dtolnay/rust-toolchain@stable
+
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v6
+ with:
+ enable-cache: true
+
+ - name: Build (release mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'release'
+ with:
+ target: aarch64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --release --strip --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Build (debug mode)
+ uses: PyO3/maturin-action@v1
+ if: inputs.build_mode == 'debug'
+ with:
+ target: aarch64-unknown-linux-gnu
+ manylinux: "2_28"
+ args: --features protoc,substrait --out dist
+ rustup-components: rust-std
+
+ - name: Archive wheels
+ uses: actions/upload-artifact@v6
+ if: inputs.build_mode == 'release'
Review Comment:
`inputs.run_wheels` is passed by callers (false for PRs) but the
non-Linux-x86_64 wheel build jobs still execute in debug mode and only skip
uploading artifacts. This likely negates a chunk of the intended PR CI speedup.
Consider gating `build-manylinux-aarch64` and `build-python-mac-win` at the job
level with `if: inputs.run_wheels` (or `if: inputs.build_mode == 'release'`) so
PRs only build the single Linux x86_64 wheel needed for pytest.
##########
.github/workflows/test.yml:
##########
@@ -79,28 +74,46 @@ jobs:
path: ~/.cargo
key: cargo-cache-${{ steps.rust-toolchain.outputs.cachekey }}-${{
hashFiles('Cargo.lock') }}
- - name: Run Clippy
- if: ${{ matrix.python-version == '3.10' && matrix.toolchain ==
'stable' }}
- run: cargo clippy --all-targets --all-features -- -D clippy::all -D
warnings -A clippy::redundant_closure
-
- - name: Install dependencies and build
+ - name: Install dependencies
uses: astral-sh/setup-uv@v7
with:
enable-cache: true
+ # Download the Linux wheel built in the build workflow
+ - name: Download pre-built Linux wheel
+ uses: actions/download-artifact@v7
+ with:
+ name: dist-manylinux-x86_64
+ path: wheels/
+
+ # Install from the pre-built wheels
+ - name: Install from pre-built wheels
+ run: |
+ set -x
+ uv venv
+ # Install documentation dependencies
Review Comment:
The comment says \"Install documentation dependencies\" but the command
shown is a generic dev sync without selecting a docs group (contrast with
`build.yml` which uses `--group docs`). Either adjust the comment to reflect
what's actually installed, or include the appropriate dependency group if docs
deps are intended here.
```suggestion
# Install development dependencies
```
##########
dev/create_license.py:
##########
@@ -22,11 +22,9 @@
import subprocess
from pathlib import Path
-subprocess.check_output(["cargo", "install", "cargo-license"])
data = subprocess.check_output(
[
- "cargo",
- "license",
+ "cargo-license",
"--avoid-build-deps",
"--avoid-dev-deps",
"--do-not-bundle",
Review Comment:
Calling `cargo-license` directly now requires the binary to already be
available on PATH. Since this script may be run outside CI, failures will be
less clear than before. Consider adding a small preflight check (e.g., via
`shutil.which('cargo-license')`) and raising a targeted error message
instructing how to install it, or falling back to invoking it via `cargo
license` so it works when installed as a cargo subcommand.
##########
.github/workflows/build.yml:
##########
@@ -92,20 +262,38 @@ jobs:
name: python-wheel-license
path: .
+ - name: Cache Cargo
+ uses: Swatinem/rust-cache@v2
+ with:
+ key: ${{ inputs.build_mode }}
+
+ - uses: astral-sh/setup-uv@v7
+ with:
+ enable-cache: true
+
- name: Install Protoc
uses: arduino/setup-protoc@v3
with:
version: "27.4"
repo-token: ${{ secrets.GITHUB_TOKEN }}
- - uses: astral-sh/setup-uv@v7
- with:
- enable-cache: true
+ - name: Install dependencies
+ run: uv sync --dev --no-install-package datafusion
- - name: Build Python package
- run: |
- uv sync --dev --no-install-package datafusion
- uv run --no-project maturin build --release --strip --features
substrait
+ # Run clippy BEFORE maturin so we can avoid rebuilding. The features
must match
+ # exactly the features used by maturin. Linux maturin builds need to
happen in a
+ # container so only run this for our mac runner.
+ - name: Run Clippy
+ if: matrix.os != 'windows-latest'
+ run: cargo clippy --no-deps --all-targets --features protoc,substrait
-- -D warnings
+
+ - name: Build Python package (release mode)
+ if: inputs.build_mode == 'release'
+ run: uv run --no-project maturin build --release --strip --features
substrait
+
+ - name: Build Python package (debug mode)
+ if: inputs.build_mode != 'release'
+ run: uv run --no-project maturin build --features substrait
Review Comment:
The comment says clippy features must match maturin exactly, but clippy runs
with `--features protoc,substrait` while maturin builds use `--features
substrait` only. This is a concrete mismatch that can cause clippy to validate
a different codepath than what is shipped on macOS/Windows. Align the features
(either add `protoc` to maturin builds here, or remove it from clippy) and keep
it consistent with the Linux wheel builds if `protoc` is required there.
```suggestion
run: uv run --no-project maturin build --release --strip --features
protoc,substrait
- name: Build Python package (debug mode)
if: inputs.build_mode != 'release'
run: uv run --no-project maturin build --features protoc,substrait
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
