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.

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

Adding npm badge to readme #71

Merged
merged 5 commits into from
Sep 1, 2017
Merged

Adding npm badge to readme #71

merged 5 commits into from
Sep 1, 2017

Conversation

romellogoodman
Copy link
Contributor

This seems pretty standard for npm packages. Thoughts?

@alexreardon
Copy link
Collaborator

Very cool! Thanks @romellogood !!!
Can you please fix up the commit history of this PR so that it is just a single commit containing the change?

Thanks!

@alexreardon alexreardon self-requested a review September 1, 2017 02:37
@alexreardon
Copy link
Collaborator

I can squash the commit in this PR. However, it would be worth cleaning up your branch so that it is the same as origin/master incase you want to make any further additions in the future :)

@alexreardon alexreardon merged commit 5f6938a into atlassian:master Sep 1, 2017
@romellogoodman
Copy link
Contributor Author

Thanks for cleaning it up for me! I'll make sure to do this in the future so its just one commit change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants