Skip to content

Commit

Permalink
fix: Force serde to always be on
Browse files Browse the repository at this point in the history
BREAKING CHANGE: `serde` feature no longer exists.
  • Loading branch information
epage committed Sep 23, 2018
1 parent 2e4ab66 commit 7f1e202
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 15 deletions.
5 changes: 1 addition & 4 deletions liquid-value/Cargo.toml
Expand Up @@ -14,13 +14,10 @@ license = "MIT"
travis-ci = { repository = "cobalt-org/liquid-rust" }
appveyor = { repository = "johannhof/liquid-rust" }

[features]
default = ["serde"]

[dependencies]
# Exposed in API
chrono = "0.4"
serde = { version = "1.0", features = ["derive"], optional = true }
serde = { version = "1.0", features = ["derive"] }

[dev-dependencies]
difference = "2.0"
Expand Down
1 change: 0 additions & 1 deletion liquid-value/src/lib.rs
Expand Up @@ -5,7 +5,6 @@
#![warn(unreachable_pub)]
#![warn(unused_extern_crates)]

#[cfg(feature = "serde")]
#[macro_use]
extern crate serde;
extern crate chrono;
Expand Down
11 changes: 4 additions & 7 deletions liquid-value/src/scalar.rs
Expand Up @@ -8,19 +8,17 @@ use chrono;
pub type Date = chrono::DateTime<chrono::FixedOffset>;

/// A Liquid scalar value
#[derive(Clone, Debug)]
#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
#[derive(Clone, Debug, Serialize, Deserialize)]
pub struct Scalar(ScalarEnum);

/// An enum to represent different value types
#[derive(Clone, Debug)]
#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
#[cfg_attr(feature = "serde", serde(untagged))]
#[derive(Clone, Debug, Serialize, Deserialize)]
#[serde(untagged)]
enum ScalarEnum {
Integer(i32),
Float(f64),
Bool(bool),
#[cfg_attr(feature = "serde", serde(with = "friendly_date"))]
#[serde(with = "friendly_date")]
Date(Date),
Str(borrow::Cow<'static, str>),
}
Expand Down Expand Up @@ -219,7 +217,6 @@ impl fmt::Display for Scalar {

const DATE_FORMAT: &str = "%Y-%m-%d %H:%M:%S %z";

#[cfg(feature = "serde")]
mod friendly_date {
use super::*;
use serde::{self, Deserialize, Deserializer, Serializer};
Expand Down
5 changes: 2 additions & 3 deletions liquid-value/src/values.rs
Expand Up @@ -18,9 +18,8 @@ type MapImpl<K, V> = BTreeMap<K, V>;
type MapImpl<K, V> = HashMap<K, V>;

/// An enum to represent different value types
#[derive(Clone, Debug)]
#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
#[cfg_attr(feature = "serde", serde(untagged))]
#[derive(Clone, Debug, Serialize, Deserialize)]
#[serde(untagged)]
pub enum Value {
/// A scalar value.
Scalar(Scalar),
Expand Down

0 comments on commit 7f1e202

Please sign in to comment.