@@ -11,7 +11,7 @@ use std::string::String;
1111use std:: { collections:: HashMap , str:: FromStr } ;
1212use tracing:: { info, warn} ;
1313
14- #[ derive( Serialize , Deserialize , Debug , PartialEq ) ]
14+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq ) ]
1515#[ serde( rename_all = "camelCase" ) ]
1616pub enum Serve {
1717 Select ( usize ) ,
@@ -52,10 +52,10 @@ impl Serve {
5252 }
5353}
5454
55- #[ derive( Serialize , Deserialize , Debug , PartialEq ) ]
55+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq ) ]
5656struct BucketRange ( ( u32 , u32 ) ) ;
5757
58- #[ derive( Serialize , Deserialize , Debug , PartialEq ) ]
58+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq ) ]
5959#[ serde( rename_all = "camelCase" ) ]
6060pub struct Distribution {
6161 distribution : Vec < Vec < BucketRange > > ,
@@ -130,7 +130,7 @@ pub struct EvalParams<'a> {
130130 segment_repo : & ' a HashMap < String , Segment > ,
131131}
132132
133- #[ derive( Serialize , Deserialize , Debug , PartialEq , Default ) ]
133+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq , Default ) ]
134134#[ serde( rename_all = "camelCase" ) ]
135135pub struct EvalDetail < T > {
136136 pub value : Option < T > ,
@@ -140,7 +140,7 @@ pub struct EvalDetail<T> {
140140 pub reason : String ,
141141}
142142
143- #[ derive( Serialize , Deserialize , Debug , PartialEq ) ]
143+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq ) ]
144144#[ serde( rename_all = "camelCase" ) ]
145145pub struct Toggle {
146146 key : String ,
@@ -296,7 +296,7 @@ struct DefaultRule {
296296 pub serve : Serve ,
297297}
298298
299- #[ derive( Serialize , Deserialize , Debug , PartialEq ) ]
299+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq ) ]
300300struct Rule {
301301 serve : Serve ,
302302 conditions : Vec < Condition > ,
@@ -317,7 +317,7 @@ impl Rule {
317317 }
318318}
319319
320- #[ derive( Serialize , Deserialize , Debug , PartialEq , Clone ) ]
320+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq , Clone ) ]
321321#[ serde( rename_all = "camelCase" ) ]
322322enum ConditionType {
323323 String ,
@@ -329,7 +329,7 @@ enum ConditionType {
329329 Unknown ,
330330}
331331
332- #[ derive( Serialize , Deserialize , Debug , PartialEq , Clone ) ]
332+ #[ derive( Serialize , Deserialize , Debug , PartialEq , Eq , Clone ) ]
333333struct Condition {
334334 r#type : ConditionType ,
335335 #[ serde( default ) ]
@@ -425,14 +425,14 @@ impl Condition {
425425 } ,
426426 None => unix_timestamp ( ) / 1000 ,
427427 } ;
428- return match predicate {
428+ match predicate {
429429 "after" => self . do_match :: < u128 > ( & c, |c, o| c. ge ( o) ) ,
430430 "before" => self . do_match :: < u128 > ( & c, |c, o| c. lt ( o) ) ,
431431 _ => {
432432 info ! ( "unknown predicate {}" , predicate) ;
433433 false
434434 }
435- } ;
435+ }
436436 }
437437
438438 fn do_match < T : FromStr > ( & self , t : & T , f : fn ( & T , & T ) -> bool ) -> bool {
0 commit comments