caas-team / sparrow

A monitoring tool to gather infrastructure network information
Apache License 2.0
6 stars 4 forks source link

Refactor: remove any typed Configuration #72

Closed puffitos closed 8 months ago

puffitos commented 8 months ago

Is there an existing feature request for this?

Problem Description

We're using any as a type in multiple places to parse the Runtime Configuration of the checks. Since we're not actually using such a feature, we should move away from using the any interface. It complicates our code and makes parsing configuration for Checks prone to errors.

Solution Description

Who can address the issue?

No response

Additional Context

No response

puffitos commented 8 months ago

I've refactored our current codebase to use some interfaces to handle Runtime Check configuration and remove the wild any usages here. We can come to this implementation or try some other once we've implemented the new Checks.