server: remove cabal flag "developer"

The only purpose was enabling the developer API by default. I don't
think that justifies a flag and CPP usage.

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3820
GitOrigin-RevId: 058c9a7b03e5e164ef88e35c42f50bae3c42b5b6
This commit is contained in:
Robert 2022-03-01 12:40:45 +01:00 committed by hasura-bot
parent 43305cd184
commit b73ce89205
4 changed files with 1 additions and 15 deletions

View File

@ -17,5 +17,4 @@ package graphql-engine
-- See:https://github.com/haskell/cabal/issues/6221
optimization: 0
documentation: false
flags: +developer

View File

@ -31,7 +31,6 @@ flags: -optimize-hasura
package graphql-engine
documentation: false
flags: +developer
-- NOTE: 'cabal new-build --enable-coverage' seems to rebuild all deps with coverage
-- which is not what we originally wanted. But building the test modules
-- themselves with coverage is actually nice for validation.

View File

@ -15,11 +15,6 @@ source-repository head
type: git
location: https://github.com/hasura/graphql-engine
flag developer
description: Enable support for development-only APIs
default: False
manual: True
flag profiling
description: Configures the project to be profiling-compatible
default: False
@ -43,8 +38,6 @@ common common-all
-- Insisting on export lists might help with compile times, and help to document modules:
-Wmissing-export-lists
if flag(developer)
cpp-options: -DDeveloperAPIs
if flag(profiling)
cpp-options: -DPROFILING

View File

@ -1,5 +1,4 @@
{-# LANGUAGE ApplicativeDo #-}
{-# LANGUAGE CPP #-}
{-# OPTIONS_GHC -O0 #-}
-- | Types and functions related to the server initialisation
@ -672,11 +671,7 @@ dangerousBooleanCollapseEnv =
)
defaultEnabledAPIs :: [API]
#ifdef DeveloperAPIs
defaultEnabledAPIs = [METADATA,GRAPHQL,PGDUMP,CONFIG,DEVELOPER]
#else
defaultEnabledAPIs = [METADATA,GRAPHQL,PGDUMP,CONFIG]
#endif
defaultEnabledAPIs = [METADATA, GRAPHQL, PGDUMP, CONFIG]
enabledAPIsEnv :: (String, String)
enabledAPIsEnv =