Remove serde feature from uuid and url dependencies (#130)
All checks were successful
Cargo CI / Build and Test (push) Successful in 1m43s
Cargo CI / Lint (push) Successful in 1m17s
Cargo CI / Build and Test (pull_request) Successful in 1m41s
Cargo CI / Lint (pull_request) Successful in 1m16s

Closes #129

Reviewed-on: #130
This commit is contained in:
Wojciech Kozlowski 2024-02-10 20:58:40 +01:00
parent fad49a48b8
commit de564eb1a0
6 changed files with 23 additions and 22 deletions

View File

@ -20,6 +20,10 @@ jobs:
RUSTFLAGS: -C instrument-coverage
steps:
- uses: actions/checkout@v3
- run: cargo build --no-default-features --all-targets
- run: cargo test --no-default-features --all-targets --no-fail-fast
- run: cargo build --all-targets
- run: cargo test --all-targets --no-fail-fast
- run: cargo build --all-features --all-targets
- run: cargo test --all-features --all-targets --no-fail-fast
- run: >-
@ -44,5 +48,7 @@ jobs:
container: docker.io/drrobot/musichoard-ci:rust-1.75
steps:
- uses: actions/checkout@v3
- run: cargo clippy --no-default-features --all-targets -- -D warnings
- run: cargo clippy --all-targets -- -D warnings
- run: cargo clippy --all-features --all-targets -- -D warnings
- run: cargo fmt -- --check

4
Cargo.lock generated
View File

@ -942,7 +942,6 @@ dependencies = [
"form_urlencoded",
"idna",
"percent-encoding",
"serde",
]
[[package]]
@ -950,9 +949,6 @@ name = "uuid"
version = "1.3.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "1674845326ee10d37ca60470760d4288a6f80f304007d92e5c53bab78c9cfd79"
dependencies = [
"serde",
]
[[package]]
name = "vec_map"

View File

@ -13,8 +13,8 @@ serde = { version = "1.0.159", features = ["derive"], optional = true }
serde_json = { version = "1.0.95", optional = true}
structopt = { version = "0.3.26", optional = true}
tokio = { version = "1.27.0", features = ["rt"], optional = true}
url = { version = "2.3.1", features = ["serde"] }
uuid = { version = "1.3.0", features = ["serde"] }
url = { version = "2.3.1" }
uuid = { version = "1.3.0" }
[dev-dependencies]
mockall = "0.11.4"

View File

@ -1,2 +1,3 @@
#[cfg(feature = "database-json")]
#![cfg(feature = "database-json")]
pub mod json;

View File

@ -1,21 +1,20 @@
#![cfg(feature = "database-json")]
#![cfg(feature = "library-beets")]
mod database;
mod library;
mod testlib;
use musichoard::MusicHoard;
#[cfg(feature = "database-json")]
use musichoard::database::json::{backend::JsonDatabaseFileBackend, JsonDatabase};
#[cfg(feature = "library-beets")]
use musichoard::library::beets::{executor::BeetsLibraryProcessExecutor, BeetsLibrary};
use musichoard::{
database::json::{backend::JsonDatabaseFileBackend, JsonDatabase},
library::beets::{executor::BeetsLibraryProcessExecutor, BeetsLibrary},
MusicHoard,
};
use crate::testlib::COLLECTION;
#[test]
#[cfg(feature = "database-json")]
#[cfg(feature = "library-beets")]
fn merge_library_then_database() {
// Acquired the lock on the beets config file. We need to own the underlying object so later we
// create a new one. This is okay as the purpose of the lock is to prevent other tests to access
@ -39,8 +38,6 @@ fn merge_library_then_database() {
}
#[test]
#[cfg(feature = "database-json")]
#[cfg(feature = "library-beets")]
fn merge_database_then_library() {
// Acquired the lock on the beets config file. We need to own the underlying object so later we
// create a new one. This is okay as the purpose of the lock is to prevent other tests to access

View File

@ -1,4 +1,5 @@
#[cfg(feature = "library-beets")]
#![cfg(feature = "library-beets")]
pub mod beets;
mod testmod;