Skip to content

Commit

Permalink
chore(grpc-asyncio): add consistent tooling
Browse files Browse the repository at this point in the history
Use the same test, lint and type checking across all our python
projects. For API, that means adding pylint.
  • Loading branch information
joekickass committed Sep 18, 2024
1 parent d86cd5f commit aef03ad
Show file tree
Hide file tree
Showing 11 changed files with 285 additions and 183 deletions.
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
name: Code QA CI
name: Python Code QA CI

on: push

jobs:
python-ci:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
python-version: ["3.8", "3.9", "3.10"]
defaults:
Expand All @@ -28,26 +27,15 @@ jobs:
run: ./docker-build.sh

- run: poetry install

- name: run pytest
run: poetry poe test

- name: run ruff
run: poetry poe lint
- name: run mypy
run: poetry poe mypy-check

rust-cargo:
name: Rust cargo tasks
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- name: Build and run tests
run: |
pushd rust/remotivelabs-broker
cargo test
popd
- name: Check format
run: |
pushd rust/remotivelabs-broker
cargo fmt --check
popd
- name: run pylint
run: poetry poe pylint

- name: run mypy
run: poetry poe mypy
24 changes: 24 additions & 0 deletions .github/workflows/rust-code-qa-ci.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
name: Rust Code QA CI

on: push

jobs:
rust-ci:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4

- uses: dtolnay/rust-toolchain@stable

- name: Build and run tests
run: |
pushd rust/remotivelabs-broker
cargo test
popd
- name: Check format
run: |
pushd rust/remotivelabs-broker
cargo fmt --check
popd
76 changes: 0 additions & 76 deletions python/remotivelabs-broker/.ruff.toml

This file was deleted.

Loading

0 comments on commit aef03ad

Please sign in to comment.