doc(filters): Internal note about the state of compact

cobalt-org · May 13, 2017 · 11424f4 · 11424f4
1 parent 36d0d2c
commit 11424f4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/filters.rs b/src/filters.rs
@@ -468,7 +468,10 @@ pub fn reverse(input: &Value, args: &[Value]) -> FilterResult {
 
 // TODO map
 
-// TODO compact
+// TODO compact removes nulls from an iterable.  For hashes, you can specify which property you
+// want to filter out if it maps to Null.
+// liquid-rust doesn't have the concept of Null.  Are there some behavior gaps we have that should
+// force us to support Null? Should we make this function a no-op?
 
 pub fn replace(input: &Value, args: &[Value]) -> FilterResult {
     if args.len() < 1 || 2 < args.len() {