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.10.0 Release #502

Merged
merged 7 commits into from
Dec 5, 2018
Merged

v6.10.0 Release #502

merged 7 commits into from
Dec 5, 2018

Conversation

Cawllec
Copy link
Contributor

@Cawllec Cawllec commented Nov 30, 2018

Goal

Updates Version & Changelog for the v6.10.0 release of: #497 #479 #495

tobyhs and others added 5 commits November 14, 2018 11:41
* Add SignalException to default ignore_classes

With #397 we receive SignalException error reports when puma shuts down,
which aren't really errors.

* Remove exit exception checks in integrations

These are now in the default ignore_classes.

* Improve IgnoredExceptions: Remove exception rejection from sidekiq integration
@Cawllec Cawllec changed the base branch from next to master December 4, 2018 16:35
@Cawllec Cawllec requested a review from a team December 4, 2018 17:40
@Cawllec
Copy link
Contributor Author

Cawllec commented Dec 4, 2018

I'm not sure why codeclimate has only detected this now, it's a pain. I'll raise a ticket to review codeclimate and possibly create a settings file to ensure it's not going to be creating false positives.

@kattrali
Copy link
Contributor

kattrali commented Dec 4, 2018

Seems ok to review linting issues on a case-by-case basis. The ones which popped for this PR had similar looking code but weren't related to each other, so I marked them as wontfix.

Copy link
Contributor

@tobyhs tobyhs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@@ -1062,7 +1071,7 @@ def gloops
expect(exception["message"]).to eq("'nil' was notified as an exception")
}
end

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How did this trailing whitespace get in here?

@Cawllec Cawllec merged commit 17a6906 into master Dec 5, 2018
@Cawllec Cawllec deleted the v6.10.0 branch December 5, 2018 11:03
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.

3 participants