Make hls-graph use explicit exports rather than module-level exports

Author: ndmitchell

hls-graph/src/Development/IDE/Graph.hs

@@ -1,4 +1,70 @@
+{-# LANGUAGE PatternSynonyms #-}
 
-module Development.IDE.Graph(module X) where
+module Development.IDE.Graph(
+    shake,
+    shakeOptions,
+    Rules, action, withoutActions, alternatives, priority, versioned,
+    Action, traced,
+    liftIO, actionOnException, actionFinally, actionBracket, actionCatch, actionRetry, runAfter,
+    ShakeException(..),
+    -- * Configuration
+    ShakeOptions(..), Rebuild(..), Lint(..), Change(..),
+    getShakeOptions, getShakeOptionsRules, getHashedShakeVersion,
+    getShakeExtra, getShakeExtraRules, addShakeExtra,
+    -- ** Command line
+    shakeArgs, shakeArgsWith, shakeArgsOptionsWith, shakeOptDescrs, addHelpSuffix,
+    -- ** Targets
+    getTargets, addTarget, withTargetDocs, withoutTargets,
+    -- ** Progress reporting
+    Progress(..), progressSimple, progressDisplay, progressTitlebar, progressProgram, getProgress,
+    -- ** Verbosity
+    Verbosity(..), getVerbosity, putVerbose, putInfo, putWarn, putError, withVerbosity, quietly,
+    -- * Running commands
+    command, command_, cmd, cmd_, unit,
+    Stdout(..), StdoutTrim(..), Stderr(..), Stdouterr(..), Exit(..), Process(..), CmdTime(..), CmdLine(..), FSATrace(..),
+    CmdResult, CmdString, CmdOption(..),
+    addPath, addEnv,
+    -- * Explicit parallelism
+    parallel, forP, par,
+    -- * Utility functions
+    copyFile', copyFileChanged,
+    readFile', readFileLines,
+    writeFile', writeFileLines, writeFileChanged,
+    removeFiles, removeFilesAfter,
+    withTempFile, withTempDir,
+    withTempFileWithin, withTempDirWithin,
+    -- * File rules
+    need, want, (%>), (|%>), (?>), phony, (~>), phonys,
+    (&%>), (&?>),
+    orderOnly, orderOnlyAction,
+    FilePattern, (?==), (<//>), filePattern,
+    needed, trackRead, trackWrite, trackAllow,
+    -- * Directory rules
+    doesFileExist, doesDirectoryExist, getDirectoryContents, getDirectoryFiles, getDirectoryDirs,
+    getDirectoryFilesIO,
+    -- * Environment rules
+    getEnv, getEnvWithDefault, getEnvError,
+    -- * Oracle rules
+    ShakeValue, RuleResult, addOracle, addOracleCache, addOracleHash, askOracle, askOracles,
+    -- * Special rules
+    alwaysRerun,
+    -- * Resources
+    Resource, newResource, newResourceIO, withResource, withResources,
+    newThrottle, newThrottleIO,
+    unsafeExtraThread,
+    -- * Cache
+    newCache, newCacheIO,
+    historyDisable, produces,
+    -- * Batching
+    needHasChanged,
+    resultHasChanged,
+    batch,
+    reschedule,
+    -- * Deprecated
+    askOracleWith,
+    deprioritize,
+    pattern Quiet, pattern Normal, pattern Loud, pattern Chatty,
+    putLoud, putNormal, putQuiet
+    ) where
 
-import Development.Shake as X
+import Development.Shake

hls-graph/src/Development/IDE/Graph/Classes.hs

@@ -1,4 +1,6 @@
 
-module Development.IDE.Graph.Classes(module X) where
+module Development.IDE.Graph.Classes(
+    Show(..), Typeable, Eq(..), Hashable(..), Binary(..), NFData(..)
+    ) where
 
-import Development.Shake.Classes as X
+import Development.Shake.Classes

hls-graph/src/Development/IDE/Graph/Database.hs

@@ -1,4 +1,14 @@
 
-module Development.IDE.Graph.Database(module X) where
+module Development.IDE.Graph.Database(
+    ShakeDatabase,
+    shakeOpenDatabase,
+    shakeWithDatabase,
+    shakeOneShotDatabase,
+    shakeRunDatabase,
+    shakeLiveFilesDatabase,
+    shakeProfileDatabase,
+    shakeErrorsDatabase,
+    shakeRunAfter
+    ) where
 
-import Development.Shake.Database as X
+import Development.Shake.Database

hls-graph/src/Development/IDE/Graph/Rule.hs

@@ -1,4 +1,24 @@
 
-module Development.IDE.Graph.Rule(module X) where
+module Development.IDE.Graph.Rule(
+    -- * Defining builtin rules
+    -- | Functions and types for defining new types of Shake rules.
+    addBuiltinRule,
+    BuiltinLint, noLint, BuiltinIdentity, noIdentity, BuiltinRun, RunMode(..), RunChanged(..), RunResult(..),
+    -- * Calling builtin rules
+    -- | Wrappers around calling Shake rules. In general these should be specialised to a builtin rule.
+    apply, apply1,
+    -- * User rules
+    -- | Define user rules that can be used by builtin rules.
+    --   Absent any builtin rule making use of a user rule at a given type, a user rule will have on effect -
+    --   they have no inherent effect or interpretation on their own.
+    addUserRule, getUserRuleList, getUserRuleMaybe, getUserRuleOne,
+    -- * Lint integration
+    -- | Provide lint warnings when running code.
+    lintTrackRead, lintTrackWrite, lintTrackAllow,
+    -- * History caching
+    -- | Interact with the non-local cache. When using the cache it is important that all
+    --   rules have accurate 'BuiltinIdentity' functions.
+    historyIsEnabled, historySave, historyLoad
+    ) where
 
-import Development.Shake.Rule as X
+import Development.Shake.Rule