Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Notification spec coverage #1001

Merged
merged 2 commits into from
Nov 22, 2017

Conversation

joerodrig
Copy link
Contributor

@joerodrig joerodrig commented Nov 22, 2017

What was the problem?

No problem. Just a little spec coverage to help get more familiarized with the code 😄

How did I fix it?

N/A

How to test it?

N/A

Review checklist

  • All new code is covered with unit tests
  • All new code follows best practices

@slaweet slaweet changed the base branch from development to 1.3.0 November 22, 2017 08:19
@slaweet slaweet self-assigned this Nov 22, 2017
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

Neat test. Thank you @joerodrig

@slaweet slaweet merged commit 65607dc into LiskArchive:1.3.0 Nov 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants