diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/Main.hs | 44 |
1 files changed, 26 insertions, 18 deletions
diff --git a/src/Main.hs b/src/Main.hs index 572dc76..7415e18 100644 --- a/src/Main.hs +++ b/src/Main.hs @@ -6,6 +6,7 @@ module Main where +import Control.Monad (unless) import Control.Monad.Identity (Identity) import Data.Aeson (eitherDecode, encode) import Data.Aeson.Encode.Pretty (encodePretty) @@ -13,33 +14,33 @@ import Data.Aeson.KeyMap (coercionToHashMap) import qualified Data.ByteString.Char8 as C8 import qualified Data.ByteString.Lazy as LB import Data.Maybe (fromMaybe) -import System.Exit (exitWith, ExitCode (..)) +import System.Exit (ExitCode (..), exitWith) import WithCli import CheckDir (recursiveCheckDir, resultIsFatal) -import LintConfig (LintConfig (..), patch) +import LintConfig (LintConfig (..), patchConfig) +import System.IO (hPutStrLn, stderr) import Types (Level (..)) import Util (printPretty) import WriteRepo (writeAdjustedRepository) -import System.IO (hPutStrLn, stderr) -- | the options this cli tool can take data Options = Options - { repository :: Maybe String + { repository :: Maybe String -- ^ path to the repository containing maps to lint - , entrypoint :: Maybe String + , entrypoint :: Maybe String -- ^ entrypoint in that repository - , json :: Bool + , json :: Bool -- ^ emit json if --json was given - , lintlevel :: Maybe Level + , lintlevel :: Maybe Level -- ^ maximum lint level to print - , pretty :: Bool + , pretty :: Bool -- ^ pretty-print the json to make it human-readable - , out :: Maybe String + , out :: Maybe String -- ^ path to write the (possibly adjusted) maps to after linting - , configFile :: Maybe FilePath + , configFile :: Maybe FilePath -- ^ path to a config file. Currently required. - , config :: Maybe (LintConfig Maybe) + , config :: Maybe (LintConfig Maybe) -- ^ a "patch" for the configuration file } deriving (Show, Generic, HasArguments) @@ -59,10 +60,8 @@ run options = do Nothing -> error "Need a config file!" Just path -> LB.readFile path >>= \res -> case eitherDecode res :: Either String (LintConfig Identity) of - Left err -> error $ "config file invalid: " <> err - Right file -> case config options of - Just p -> pure (patch file p) - Nothing -> pure file + Left err -> error $ "config file invalid: " <> err + Right file -> pure (patchConfig file (config options)) lints <- recursiveCheckDir lintconfig repo entry @@ -72,11 +71,20 @@ run options = do else printPretty (level, lints) case out options of - Just outpath -> writeAdjustedRepository lintconfig repo outpath lints - >>= exitWith Nothing -> exitWith $ case resultIsFatal lintconfig lints of False -> ExitSuccess - True -> ExitFailure 1 + True -> ExitFailure 1 + Just outpath -> do + c <- writeAdjustedRepository lintconfig repo outpath lints + unless (json options) $ + case c of + ExitFailure 1 -> putStrLn "\nMap failed linting!" + ExitFailure 2 -> putStrLn "\nOutpath already exists, not writing anything." + _ -> pure () + exitWith c + + + -- | haskell's many string types are FUN … printLB :: LB.ByteString -> IO () |