Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V6 #320

Closed
wants to merge 94 commits into from
Closed

V6 #320

wants to merge 94 commits into from

Conversation

snmaynard
Copy link
Contributor

This is a fairly big rewrite/change for the ruby notifier.

Its not complete but I wanted to deprecate lots of old interfaces for the ruby notifier.

Changes

-config.ignore_classes now no longer accepts strings

  • Bugsnag.notify_or_ignore removed - now call notify directly
  • Bugsnag.auto_notify removed - now call notify directly
  • config.delay_with_resque removed
  • Bugsnag::Notification now Bugsnag::Report
  • notify now only supports the block, you can no longer send in overrides hash
  • config.use_ssl removed, now set protocol in config.endpoint
  • config.vendor_paths removed (Should we leave this in for people that locally install gemfile?)
  • after_notify_callbacks removed
  • Capistrano support removed - will go into a dedicated gem
  • Deploy support removed, will go in the capistrano gem
  • params_filters renamed to meta_data_filters to be clearer
  • config.debug removed and instead use the logger directly

Probably more besides. This cannot be merged until the capistrano gem exists.

@keeganlow keeganlow changed the title V5 V6 Aug 23, 2016
@kattrali kattrali mentioned this pull request Sep 5, 2016
@kattrali kattrali mentioned this pull request Oct 30, 2017
2 tasks
@kattrali
Copy link
Contributor

Closing for #385 (resolves conflicts).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants