search

Patrick-Kladek / Callisto

Parse Clang Static Analyzer messages and pass them to Slack
MIT License
21 stars 3 forks source link

Improve Project Structure #6

Open schwmi opened 4 years ago

schwmi commented 4 years ago

Currently we have one CommandLine Target and one UnitTest target. To support unit testing, all source code has to be added to both targets.

Better structure would be to move the Calisto code into a framework target which is unit-testable and have only a very compact main.swift in the command line target, which executes the framework.

Patrick-Kladek commented 4 years ago

Thats would be a great idea. Because it's a command line target I would strongly suggest to be a static library otherwise we end up with a framework and an executable which both needs to be in sync.