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.

Sign up for GitHub

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

Jump to bottom

unify error handling #1051

Merged
merged 1 commit into from
Feb 9, 2017
Merged

unify error handling #1051

merged 1 commit into from
Feb 9, 2017

Conversation

guwirth
Copy link
Collaborator

@guwirth guwirth commented Feb 9, 2017

@guwirth guwirth added the bug label Feb 9, 2017
@guwirth guwirth added this to the 0.9.7 milestone Feb 9, 2017
@guwirth guwirth self-assigned this Feb 9, 2017
@guwirth guwirth requested a review from jmecosta February 9, 2017 07:57
jmecosta
jmecosta approved these changes Feb 9, 2017
View reviewed changes
Copy link
Member

@jmecosta jmecosta left a comment

Choose a reason for hiding this comment

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

Still had those missing! Good. Approved

@guwirth
unify error handling
c31d3f7 
- all sensors should continue if 'sonar.cxx.errorRecoveryEnabled = true'
- 'validateRecovery' method should start with lower case
- close #1050
@guwirth
Copy link
Collaborator Author

guwirth commented Feb 9, 2017

@jmecosta thx for reviewing

@guwirth guwirth merged commit 1e32dd5 into SonarOpenCommunity:master Feb 9, 2017
@guwirth guwirth deleted the error-handling branch February 17, 2017 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmecosta jmecosta jmecosta approved these changes

Assignees

@guwirth guwirth

Labels
bug
Milestone
0.9.7
Development

Successfully merging this pull request may close these issues.

CxxXunitSensor exception handling a boost report
2 participants
@guwirth @jmecosta