Deprecations for 3.0

[tombruijn]

Deprecated in 2.x

Methods and arguments to be removed for version 3.0.

• lib/appsignal.rb - #start_logger
  • path_arg argument is no longer supported. Please remove

    appsignal-ruby/lib/appsignal.rb

    Lines 257 to 259 in 4eed259

    	if path_arg
    	logger.info('Setting the path in start_logger has no effect anymore, set it in the config instead')
    	end

• lib/appsignal.rb - .is_ignored_error?
  • Deprecated in PR Remove ignored errors #224 -- remove in 3.0
• lib/appsignal.rb - .is_ignored_action?
  • Deprecated in PR Remove ignored errors #224 -- remove in 3.0
• lib/appsignal/auth_check.rb
  • logger argument - deprecated in Deprecate unused AuthCheck logger argument #245
• lib/appsignal/transaction.rb
  • Transaction#store - deprecated in Set the app environment using the APPSIGNAL_APP_ENV env var #33 Never actually merged. Transaction#store is still used in the mongo_ruby_driver integration.
• lib/appsignal/config.rb
  • backwards compatability for older config keys: Extract backwards compatibility logic to a method #359
• lib/appsignal/event_formatter.rb
  • deprecated in Refactor event_formatter registration code #397

  • appsignal-ruby/lib/appsignal/event_formatter.rb

    Lines 88 to 97 in 0843767

    	def register_deprecated_formatter(name)
    	Appsignal.logger.warn(
    	"Formatter for '#{name}' is using a deprecated registration " \
    	"method. This event formatter will not be loaded. " \
    	"Please update the formatter according to the documentation at: " \
    	"https://docs.appsignal.com/ruby/instrumentation/event-formatters.html"
    	)
    	deprecated_formatter_classes[name] = self
    	end

• appsignal notify_of_deploy - Print deprecation warning for notify_of_deploy task #414
  • Deprecated in Deprecate notify_of_deploy command #545
• lib/appsignal/integrations/resque.rb instrumentation - PR Rework Resque and Resque ActiveJob integrations #630
• lib/appsignal/integrations/resque_active_job.rb instrumentation - PR Rework Resque and Resque ActiveJob integrations #630

Add more deprecations here (and explicitly deprecate them in the code) once you find them.

To be deprecated before v3.0 (moved to #667)

• lib/appsignal.rb - .monitor_transaction
  • create a better alternative for this helper
• lib/appsignal/auth_check.rb
  • merge #perform and #perform_with_result
• Capistrano integration's deploy logic.
  • Prefer the revision config option instead.

[tombruijn]

Closing. PR #685 was merged.