Skip to content

Commit

Permalink
Combined structured output and updated default rule clause name to in…
Browse files Browse the repository at this point in the history
…clude file name (aws-cloudformation#360)

* Populated filename in the output

* Structured combine

* Changed FileData into DataFile and handle error differently

* Resolved lifetime issue with FileReport combine method

* Updated status and method

* Refactored to use existing DataFile struct

* Changed FileData into DataFile and handle error differently

* Refactored to use existing DataFile struct

* Merged file report

* Interim commit for structured

* Resolved unit tests

* Temporary commit for default rule names

* Working prototype for formatting issue

---------

Co-authored-by: Akshay Rane <raneaks@amazon.com>
  • Loading branch information
2 people authored and joshfried-aws committed Jun 30, 2023
1 parent aca8d2f commit 2661a29
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion guard/src/commands/validate/structured.rs
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ impl<'eval> StructuredEvaluator<'eval> {
if let Status::FAIL = eval_rules_file(rule, &mut root_scope, Some(&each.name))? {
self.exit_code = 19;
}

let root_record = root_scope.reset_recorder().extract();
let report = simplifed_json_from_root(&root_record)?;
file_report.combine(report);
Expand Down

0 comments on commit 2661a29

Please sign in to comment.