Don't mark manual rules without Rego as missing

[wwwil]

Rules that are marked as manual should not be flagged as missing if they don't have Rego implementation as they are not supposed to.

Closes #76.

Signed-off-by: wwwil wwwil.squires@gmail.com

[jetstack-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wwwil

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [wwwil]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[j-fuentes]

+1 to this change, but I don't think this closes #76.

Currently api.Report does not contain Manual, which means it is impossible to tell if a rule failed or it requires a manual check.

I suggest either filtering out manual rules from the report or surfacing the Manual.

[wwwil]

Currently api.Report does not contain Manual

The api. ReportRule has a Manual field: https://github.com/jetstack/preflight/blob/master/api/report.go#L81

I think the next issue is that we need to update the web UI to show manual rules differently, or at least highlight that they are manual.

[j-fuentes]

/lgtm