Feature/compilo vm #93
Tests.yml
on: pull_request
check_compilation
1m 42s
check_style
9s
run_tests
3m 0s
Annotations
10 errors and 14 warnings
check_style
HLint failed with status: 1. Warning (7), Suggestion (20)
|
run_tests:
LobsterLang/test/CompilerSpec.hs#L28
Data constructor not in scope: PushS :: String -> Instruction
|
run_tests:
LobsterLang/test/CompilerSpec.hs#L34
Data constructor not in scope: PushS :: String -> Instruction
|
run_tests:
LobsterLang/test/CompilerSpec.hs#L42
Data constructor not in scope: PushS :: String -> Instruction
|
run_tests:
LobsterLang/test/CompilerSpec.hs#L75
Data constructor not in scope: PushS :: String -> Instruction
|
run_tests:
LobsterLang/test/VmSpec.hs#L22
Data constructor not in scope: Subtract :: Operator
|
run_tests:
LobsterLang/test/VmSpec.hs#L24
Data constructor not in scope: Multiply :: Operator
|
run_tests:
LobsterLang/test/VmSpec.hs#L26
Data constructor not in scope: Divide :: Operator
|
run_tests:
LobsterLang/test/VmSpec.hs#L28
Data constructor not in scope: Divide :: Operator
|
run_tests
Process completed with exit code 2.
|
check_style:
LobsterLang/src/Parse.hs#L51
Suggestion in Parser in module Parse: Use newtype instead of data ▫︎ Found: "data Parser a\n = Parser {runParser :: Position\n -> String -> Either String (a, String, Position)}" ▫︎ Perhaps: "newtype Parser a\n = Parser {runParser :: Position\n -> String -> Either String (a, String, Position)}" ▫︎ Note: decreases laziness
|
check_style:
LobsterLang/src/Parse.hs#L212
Warning in parseElem in module Parse: Use const ▫︎ Found: "\\ x _ -> x" ▫︎ Perhaps: "const"
|
check_style:
LobsterLang/src/Parse.hs#L309
Suggestion in parseListElem in module Parse: Redundant bracket ▫︎ Found: "case (parseOthers parser pos' s') of\n Left err -> Left err\n Right (res', s'', pos'') -> Right (res : res', s'', pos'')" ▫︎ Perhaps: "case parseOthers parser pos' s' of\n Left err -> Left err\n Right (res', s'', pos'') -> Right (res : res', s'', pos'')"
|
check_style:
LobsterLang/src/Parse.hs#L317
Suggestion in parseListElem in module Parse: Redundant bracket ▫︎ Found: "case (parseOthers parser pos'' s'') of\n Left err -> Left err\n Right (res', s''', pos''') -> Right (res : res', s''', pos''')" ▫︎ Perhaps: "case parseOthers parser pos'' s'' of\n Left err -> Left err\n Right (res', s''', pos''') -> Right (res : res', s''', pos''')"
|
check_style:
LobsterLang/src/Parse.hs#L421
Suggestion in parseDefineFn in module Parse: Redundant bracket ▫︎ Found: "parseCmpString \"fn\" *> (Parser defineFn)" ▫︎ Perhaps: "parseCmpString \"fn\" *> Parser defineFn"
|
check_style:
LobsterLang/src/Parse.hs#L460
Suggestion in parseCond in module Parse: Redundant bracket ▫︎ Found: "((AST.Cond res res' Nothing), s'', pos'')" ▫︎ Perhaps: "(AST.Cond res res' Nothing, s'', pos'')"
|
check_style:
LobsterLang/src/Parse.hs#L461
Suggestion in parseCond in module Parse: Redundant bracket ▫︎ Found: "((AST.Cond res res' (Just res'')), s''', pos''')" ▫︎ Perhaps: "(AST.Cond res res' (Just res''), s''', pos''')"
|
check_style:
LobsterLang/src/Vm.hs#L42
Warning in module Vm: Redundant bracket ▫︎ Found: "(0)" ▫︎ Perhaps: "0"
|
check_style:
LobsterLang/src/Vm.hs#L52
Warning in module Vm: Redundant bracket ▫︎ Found: "(0)" ▫︎ Perhaps: "0"
|
check_style:
LobsterLang/src/Vm.hs#L62
Warning in module Vm: Redundant bracket ▫︎ Found: "(0)" ▫︎ Perhaps: "0"
|
run_tests:
LobsterLang/test/AstEvalSpec.hs#L1
The export item ‘module AstEvalSpec’ is missing an export list
|
run_tests:
LobsterLang/test/AstOptimizerSpec.hs#L1
The export item ‘module AstOptimizerSpec’ is missing an export list
|
run_tests:
LobsterLang/test/ParserSpec.hs#L1
The export item ‘module ParserSpec’ is missing an export list
|
run_tests:
LobsterLang/test/ParserSpec.hs#L12
The import of ‘Parse’ is redundant
|