Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Comprehensive tests for tracing transactions #791

Merged
merged 6 commits into from Mar 20, 2016
Merged

Comprehensive tests for tracing transactions #791

merged 6 commits into from Mar 20, 2016

Conversation

gavofyork
Copy link
Contributor

Fixed some behaviour - failed actions are still logged (though the result is accordingly None).

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Mar 20, 2016
@gavofyork gavofyork added this to the 1.1 Civility milestone Mar 20, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 20, 2016
arkpar added a commit that referenced this pull request Mar 20, 2016
Comprehensive tests for tracing transactions
@arkpar arkpar merged commit 8a5aa35 into master Mar 20, 2016
@gavofyork gavofyork mentioned this pull request Mar 20, 2016
3 tasks
@gavofyork gavofyork modified the milestone: 1.1 Civility Mar 21, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants