Skip to content

Commit

Permalink
dashboard/app: fix unused variable warning
Browse files Browse the repository at this point in the history
isAppEngineTest is meant to be used in prod configs.
  • Loading branch information
dvyukov committed Feb 17, 2019
1 parent 3636a62 commit 6a4471c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions dashboard/app/aetest.go
Expand Up @@ -5,4 +5,6 @@

package dash

// isAppEngineTest is meant to be used in prod config to either
// load the config or just check its correctness.
const isAppEngineTest = true
2 changes: 1 addition & 1 deletion dashboard/app/config.go
Expand Up @@ -161,7 +161,7 @@ var config *GlobalConfig

func init() {
// Prevents gometalinter from considering everything as dead code.
if false {
if false && isAppEngineTest {
installConfig(nil)
}
}
Expand Down

0 comments on commit 6a4471c

Please sign in to comment.