Tyrrrz / CliFx

Class-first framework for building command-line interfaces
MIT License
1.5k stars 61 forks source link

Refactor CommandSchemaAnalyzer into separate analyzers #89

Closed Tyrrrz closed 3 years ago

Tyrrrz commented 3 years ago

Currently, CommandSchemaAnalyzer is one bloated class with about 15 checks. The corresponding tests, CommandSchemaAnalyzerTests are even more bloated. This makes maintaining and adding new checks quite difficult.

The suggestion is to refactor each check in its own separate analyzer and break the tests up correspondingly as well.