Skip to content

Commit

Permalink
Allow to use both async-channel and futures channels via feature flags (
Browse files Browse the repository at this point in the history
#48)

* Allow to select between channels implementations

* Fix tests

* Add futures-channel tests in CI

* Fix check job

* feat: send signals in parallel (#51)

Co-authored-by: Bernhard Schuster <bernhard@ahoi.io>

---------

Co-authored-by: Bernhard Schuster <bernhard@ahoi.io>
  • Loading branch information
vstakhov and drahnr authored Jul 18, 2023
1 parent 06b116a commit 137f739
Show file tree
Hide file tree
Showing 11 changed files with 528 additions and 198 deletions.
47 changes: 46 additions & 1 deletion .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,30 @@ jobs:
uses: actions-rs/cargo@v1.0.3
with:
command: check
args: --all-targets --all-features --workspace
args: --all-targets --workspace

build-futures:
name: Cargo check futures channel
runs-on: ubuntu-latest
steps:
- name: Checkout sources
uses: actions/checkout@v2

- name: Install Rust stable toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true

- name: Rust Cache
uses: Swatinem/rust-cache@v1.3.0

- name: Build
uses: actions-rs/cargo@v1.0.3
with:
command: check
args: --no-default-features --features futures_channel --manifest-path metered-channel/Cargo.toml

fmt:
name: Cargo fmt
Expand Down Expand Up @@ -85,6 +108,28 @@ jobs:
with:
command: test
args: --all-targets --workspace
tests-futures:
name: Cargo test with futures
runs-on: ubuntu-latest
steps:
- name: Checkout sources
uses: actions/checkout@v2

- name: Install Rust stable toolchain
uses: actions-rs/toolchain@v1
with:
profile: minimal
toolchain: stable
override: true

- name: Rust Cache
uses: Swatinem/rust-cache@v1.3.0

- name: Cargo test
uses: actions-rs/cargo@v1.0.3
with:
command: test
args: --no-default-features --features futures_channel --manifest-path metered-channel/Cargo.toml

clippy:
name: Cargo clippy
Expand Down
Loading

0 comments on commit 137f739

Please sign in to comment.