Missing indentation in YAML created by OSVVM

OSVVM / OSVVM-Scripts

OSVVM project simulation scripts. Scripts are tedious. These scripts simplify the steps to compile your project for simulation

Other

10 stars 14 forks source link

Missing indentation in YAML created by OSVVM #14

Closed Paebbels closed 2 years ago

Paebbels commented 2 years ago

The generated YAML output of OSVVM has a missing indentation level before echt testcase.

Corrected Output as diff:

TestSuites: 
  - Name: Axi4Lite
    TestCases:
-    - TestCaseName: TbAxi4_BasicReadWrite
-      Name: "TbAxi4_BasicReadWrite"
-      Status: PASSED
+      - TestCaseName: TbAxi4_BasicReadWrite
+        Name: "TbAxi4_BasicReadWrite"
+        Status: PASSED

I haven't checks on functional coverage YAML files.

Paebbels commented 2 years ago

Was this fixed in the last release?

JimLewis commented 2 years ago

Is this pedantic or Is it breaking something? TCL YAML reader is reading it just fine. If it is breaking the EDAA reader, I will prioritize it higher.

Paebbels commented 2 years ago

Some reader do read it, but the syntax is not correct.

So priority is low, but should be fixed.

JimLewis commented 2 years ago

Agreed. It is on my list, but lower than the large list of other things that must be done. But it will get done as sometimes it just feels good to get things off the list.

JimLewis commented 2 years ago

Fixed in next release