Skip to content

Commit a960612

Browse files
pftgclaude
andcommitted
fix: remove ActiveSupport dependency from fail_if_new default
ENV["CI"].present? crashes when ActiveSupport core extensions aren't loaded yet. Discovered when integrating with jetthoughts.github.io which loads the gem before Rails. Use plain Ruby nil/empty check. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 861183f commit a960612

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

lib/capybara_screenshot_diff.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ def screenshot_area_abs
6363
module Diff
6464
mattr_accessor(:delayed) { true }
6565
mattr_accessor :area_size_limit
66-
mattr_accessor(:fail_if_new) { ENV["CI"].present? }
66+
mattr_accessor(:fail_if_new) { !ENV["CI"].nil? && !ENV["CI"].empty? }
6767
mattr_accessor(:fail_on_difference) { true }
6868
mattr_accessor :color_distance_limit
6969
mattr_accessor(:enabled) { true }

0 commit comments

Comments
 (0)