Improve BacktraceCleaner code example [ci-skip]

byroot · byroot · commit 869c7bf094c0 · 2024-06-19T09:16:30.000+02:00
With newer rubies, removing a prefix is much simpler.
diff --git a/activesupport/lib/active_support/backtrace_cleaner.rb b/activesupport/lib/active_support/backtrace_cleaner.rb
@@ -18,7 +18,7 @@ module ActiveSupport
   #
   #   bc = ActiveSupport::BacktraceCleaner.new
   #   root = "#{Rails.root}/"
-  #   bc.add_filter   { |line| line.start_with?(root) ? line.from(root.size) : line } # strip the Rails.root prefix
+  #   bc.add_filter   { |line| line.delete_prefix(root) } # strip the Rails.root prefix
   #   bc.add_silencer { |line| /puma|rubygems/.match?(line) } # skip any lines from puma or rubygems
   #   bc.clean(exception.backtrace) # perform the cleanup
   #

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ module ActiveSupport`
`18`	`18`	`#`
`19`	`19`	`# bc = ActiveSupport::BacktraceCleaner.new`
`20`	`20`	`# root = "#{Rails.root}/"`
`21`		`- # bc.add_filter { \|line\| line.start_with?(root) ? line.from(root.size) : line } # strip the Rails.root prefix`
	`21`	`+ # bc.add_filter { \|line\| line.delete_prefix(root) } # strip the Rails.root prefix`
`22`	`22`	`# bc.add_silencer { \|line\| /puma\|rubygems/.match?(line) } # skip any lines from puma or rubygems`
`23`	`23`	`# bc.clean(exception.backtrace) # perform the cleanup`
`24`	`24`	`#`