cli/reportgen

aderyn_core/src/detect/high/high_level_calls_in_loop.rs

@@ -81,9 +81,9 @@ impl IssueDetector for HighLevelCallsInLoopDetector {
     }
 
     fn description(&self) -> String {
-        String::from("One of the dangers of having high level calls or an `address.transfer()` inside a loop is that\
-            if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended\
-            to convert high level calls into low level calls so that we get a `bool success` instead of an automatic\
+        String::from("One of the dangers of having high level calls or an `address.transfer()` inside a loop is that \
+            if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended \
+            to convert high level calls into low level calls so that we get a `bool success` instead of an automatic \
             revert on failure.")
     }
 

reports/ccip-functions-report.md

@@ -490,7 +490,7 @@ Solidity does initialize variables by default when you declare them, however it'
 
 ## H-4: Loop contains high level calls to other contracts and/or eth is transferred.
 
-One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure.
+One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure.
 
 <details><summary>10 Found Instances</summary>
 

reports/report.json

@@ -1801,7 +1801,7 @@
       },
       {
         "title": "Loop contains high level calls to other contracts and/or eth is transferred.",
-        "description": "One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure.",
+        "description": "One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure.",
         "detector_name": "high-level-calls-in-loop",
         "instances": [
           {

reports/report.md

@@ -1792,7 +1792,7 @@ A deletion in a structure containing a mapping will not delete the mapping. The
 
 ## H-33: Loop contains high level calls to other contracts and/or eth is transferred.
 
-One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure.
+One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure.
 
 <details><summary>4 Found Instances</summary>
 

reports/report.sarif

@@ -2662,7 +2662,7 @@
             }
           ],
           "message": {
-            "text": "One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure."
+            "text": "One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure."
           },
           "ruleId": "high-level-calls-in-loop"
         },

reports/sablier-aderyn-toml-nested-root.md

@@ -68,7 +68,7 @@ This report was generated by [Aderyn](https://github.com/Cyfrin/aderyn), a stati
 
 ## H-1: Loop contains high level calls to other contracts and/or eth is transferred.
 
-One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure.
+One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure.
 
 <details><summary>2 Found Instances</summary>
 

reports/templegold-report.md

@@ -525,7 +525,7 @@ A deletion in a structure containing a mapping will not delete the mapping. The
 
 ## H-9: Loop contains high level calls to other contracts and/or eth is transferred.
 
-One of the dangers of having high level calls or an `address.transfer()` inside a loop is thatif a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommendedto convert high level calls into low level calls so that we get a `bool success` instead of an automaticrevert on failure.
+One of the dangers of having high level calls or an `address.transfer()` inside a loop is that if a single call fails, then the whole transaction is reverted which maybe undesirable. It's recommended to convert high level calls into low level calls so that we get a `bool success` instead of an automatic revert on failure.
 
 <details><summary>17 Found Instances</summary>