-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add library files #1
Conversation
@ajeetdsouza Can you run the Haskell files through stylish-haskell? Just do |
src/CI.hs
Outdated
{-# LANGUAGE OverloadedStrings #-} | ||
{-# LANGUAGE TemplateHaskell #-} | ||
|
||
module CI ( Types.CI(..), isCI, getCI ) where |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exports in new line
module CI
( Types.CI (..)
, ----
, -----
) where
src/CI.hs
Outdated
import Data.Maybe ( isJust ) | ||
import qualified Data.Text as T | ||
|
||
import System.Environment ( getEnvironment ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
follow import style mentioned here
src/CI/Types.hs
Outdated
|
||
instance Aeson.ToJSON VendorEnv where | ||
toJSON val = case val of | ||
VendorEnvString key -> Aeson.toJSON key |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
key
-> name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.