Skip to content

Commit

Permalink
Merge pull request #36 from jcelliott/fix_mix_env_undefined
Browse files Browse the repository at this point in the history
Remove runtime calls to Mix.env/0
  • Loading branch information
hahuang65 committed Jul 15, 2016
2 parents 84667b1 + c9ef17d commit a8b32bc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
3 changes: 2 additions & 1 deletion lib/beaker/time_series/aggregator.ex
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
defmodule Beaker.TimeSeries.Aggregator do
@name :beaker_time_series_aggregator
@default_interval 60
@env Mix.env

@moduledoc """
`Beaker.TimeSeries.Aggregator` is the aggregation framework for time series. It's purposed mostly for internal use only.
Expand Down Expand Up @@ -84,7 +85,7 @@ defmodule Beaker.TimeSeries.Aggregator do

@doc false
def init(:ok) do
unless Mix.env == :test do
unless @env == :test do
:timer.send_interval(@default_interval * 1000, :schedule_aggregation)
end
{:ok, Beaker.Time.last_full_minute}
Expand Down
5 changes: 0 additions & 5 deletions lib/web/helper.ex
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,6 @@ defmodule Beaker.Web.Helper do

@colors ~w(yellow orange red magenta violet blue cyan green)

def autorefresh_interval, do: autorefresh_interval(Mix.env)
def autorefresh_interval(:dev), do: 5000
def autorefresh_interval(:prod), do: 30000
def autorefresh_interval(:test), do: nil

def random_color do
:random.seed(:os.timestamp)

Expand Down

0 comments on commit a8b32bc

Please sign in to comment.