Merge pull request #485 from airbrake/performance-stats-true

config: enable performance_stats by default
airbrake · Jun 25, 2019 · 9426019 · 9426019
2 parents 738162b + 2b5ef9f
commit 9426019
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,8 @@ Airbrake Ruby Changelog
 * Fixed `AsyncSender` returning `nil` instead of a rejected promise when it
   reaches its capacity
   ([#484](https://github.com/airbrake/airbrake-ruby/pull/484))
+* Enabled `performance_stats` by default
+  ([#485](https://github.com/airbrake/airbrake-ruby/pull/485))
 
 ### [v4.4.0][v4.4.0] (May 9, 2019)
 

diff --git a/README.md b/README.md
@@ -362,7 +362,7 @@ all accounts. Please <a href="mailto:support@airbrake.io?subject=Perfomance Moni
 
 Configures performance statistics collection (routes, SQL queries) This
 statistics is displayed on the Performance tab of your project. By default, it's
-disabled.
+enabled.
 
 The statistics is sent via:
 
@@ -371,7 +371,7 @@ The statistics is sent via:
 
 ```ruby
 Airbrake.configure do |c|
-  c.performance_stats = true
+  c.performance_stats = false
 end
 ```
 

diff --git a/lib/airbrake-ruby/config.rb b/lib/airbrake-ruby/config.rb
@@ -130,7 +130,7 @@ def initialize(user_config = {})
       )
 
       self.versions = {}
-      self.performance_stats = false
+      self.performance_stats = true
       self.performance_stats_flush_period = 15
 
       merge(user_config)

diff --git a/spec/config_spec.rb b/spec/config_spec.rb
@@ -19,7 +19,7 @@
   its(:timeout) { is_expected.to be_nil }
   its(:blacklist_keys) { is_expected.to be_empty }
   its(:whitelist_keys) { is_expected.to be_empty }
-  its(:performance_stats) { is_expected.to be_falsey }
+  its(:performance_stats) { is_expected.to eq(true) }
   its(:performance_stats_flush_period) { is_expected.to eq(15) }
 
   describe "#new" do