Merge #29

bors[bot] · jaysonsantos · web-flow · commit f7467a3b12f1 · 2022-06-22T07:58:49.000Z
29: fix: Do not needs-hash annotation when it is already supplied r=jaysonsantos a=jaysonsantos



Co-authored-by: Jayson Reis &lt;santosdosreis@gmail.com&gt;
diff --git a/src/maps.rs b/src/maps.rs
@@ -50,7 +50,9 @@ pub fn generate_output_map(
     match annotations {
         Metadata::String(_) => panic!("{} shoult not be a string at this point", ANNOTATIONS_KEY),
         Metadata::HashMap(h) => {
-            h.insert(ANNOTATION_NEEDS_HASH.into(), "true".into());
+            if !h.contains_key(ANNOTATION_NEEDS_HASH) {
+                h.insert(ANNOTATION_NEEDS_HASH.into(), "true".into());
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,9 @@ pub fn generate_output_map(`
`50`	`50`	`match annotations {`
`51`	`51`	`Metadata::String(_) => panic!("{} shoult not be a string at this point", ANNOTATIONS_KEY),`
`52`	`52`	`Metadata::HashMap(h) => {`
`53`		`- h.insert(ANNOTATION_NEEDS_HASH.into(), "true".into());`
	`53`	`+ if !h.contains_key(ANNOTATION_NEEDS_HASH) {`
	`54`	`+ h.insert(ANNOTATION_NEEDS_HASH.into(), "true".into());`
	`55`	`+ }`
`54`	`56`	`}`
`55`	`57`	`}`
`56`	`58`