HL7 / fhir-ig-publisher

Source code for the IG publisher
Apache License 2.0
68 stars 56 forks source link

Feature: More helpful QA errors-only page #957

Open citizenrich opened 1 day ago

citizenrich commented 1 day ago

Right now the report qa.min.html shows all information, warnings, and errors. When troubleshooting a large number of build errors it would be helpful if the 'See Errors Only' actually only showed errors, rather than nearly all the same information as the full qa report.

grahamegrieve commented 1 day ago

so I thought that the view only shows errors; that's how I wrote it and how it's described. But unaccountably, the code says:

for (ValidationMessage vm : uniqueErrors) {
      if (vm.getLevel() != IssueSeverity.INFORMATION) {
        return true;
      }
    }

Apparently I wrote it like that, but it should just be errors, not errors and warnings