Skip to content

Conversation

ami
Copy link
Contributor

@ami ami commented Apr 18, 2015

No description provided.

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

...examples of how to...

@ami
Copy link
Contributor Author

ami commented Apr 20, 2015

@elnaz updated!

@elnaz
Copy link
Contributor

elnaz commented Apr 20, 2015

LGTM. I have to merge the bank verify endpoint PR before this so I'll take care of merging this too.

@leore
Copy link
Contributor

leore commented Apr 20, 2015

Other PR is in. I think if we rebase this will now pass.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.52%) to 100.0% when pulling d0979b9 on ami:patch-5 into 2418253 on lob:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.52%) to 100.0% when pulling d0979b9 on ami:patch-5 into 2418253 on lob:master.

@leore
Copy link
Contributor

leore commented Apr 20, 2015

LGTM (merge away) @pon can do you want to show how to publish

elnaz added a commit that referenced this pull request Apr 20, 2015
fix(readme): standardize format + link to docs
@elnaz elnaz merged commit 8d51a2f into lob:master Apr 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants