The default linter severity should be used when severity is not defined in linter config file

[mbarre]

Hi Sualeh,

When we use xml config file without specifying the linter severity, MEDIUM severity is used instead of the default linter severity.

Ex:

public class LinterTableWithNoPrimaryKey extends BaseLinter {
    public LinterTableWithNoPrimaryKey(){
        setSeverity(LintSeverity.high);
    }
 ...
}

Here default is high.

If I use it in config file without specify the severity

<linter id="io.github.mbarre.schemacrawler.tool.linter.LinterTableWithNoPrimaryKey">
        <run>true</run>
        <table-exclusion-pattern><![CDATA[.*databasechangelog*]]></table-exclusion-pattern>
    </linter>

It will be report as lint with MEDIUM severity instead of HIGH.

It would be nice if the default linter severity were used when not defined in config file.

What do you think ?

[mbarre]

Thanks Sualeh ! :+1: