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

Added time stamp to secondary events #709

Merged
merged 1 commit into from Apr 18, 2017

Conversation

Projects
None yet
3 participants
@janhahne
Contributor

janhahne commented Apr 12, 2017

This fixes #707. I suggest @jakobj and @weidel-p as reviewers.

@weidel-p

This comment has been minimized.

Show comment
Hide comment
@weidel-p

weidel-p Apr 13, 2017

Contributor

@janhahne thank you for fixing this issue.

This PR is very short, contains a regressiontest and looks fine to me. 👍

Contributor

weidel-p commented Apr 13, 2017

@janhahne thank you for fixing this issue.

This PR is very short, contains a regressiontest and looks fine to me. 👍

@heplesser heplesser self-requested a review Apr 18, 2017

@heplesser heplesser merged commit 81b640f into nest:master Apr 18, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@janhahne janhahne deleted the janhahne:fix_707 branch Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment