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

Pinterest and tumblr #30

Merged
merged 5 commits into from
Jun 21, 2018
Merged

Pinterest and tumblr #30

merged 5 commits into from
Jun 21, 2018

Conversation

prateekrastogi
Copy link
Contributor

  • Added pinterest and tumblr.

  • Also, added hashtag field in twitter, and title field in reddit.

@coveralls
Copy link

coveralls commented Jun 21, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 4a23894 on prateekrastogi:PinterestAndTumblr into 5d4d044 on avdeev:master.

Copy link
Owner

@avdeev avdeev left a comment

Choose a reason for hiding this comment

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

@prateekrastogi Thanks!
Plz, see my comments

README.md Outdated
### `tumblr(options)`

Share on Tumblr

Copy link
Owner

Choose a reason for hiding this comment

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

attach a link to the documentation, please

Copy link
Contributor Author

@prateekrastogi prateekrastogi Jun 21, 2018

Choose a reason for hiding this comment

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

attached link to documentation of both in this push

README.md Outdated
### `pinterest(options)`

Share on Pinterest

Copy link
Owner

Choose a reason for hiding this comment

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

attach a link to the documentation, please

package.json Outdated
@@ -16,7 +16,7 @@
"main": "dist/vanilla-sharing.umd.js",
"module": "dist/vanilla-sharing.esm.js",
"scripts": {
"precommit": "lint-staged",
"precommit": "npm run lint --",
Copy link
Owner

Choose a reason for hiding this comment

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

why?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At first, when i was trying to make my first commit on windows machine, npm was showing an error with change as suggested fix. The issue persisted on two different windows machine at home and work. So, I changed it. Now, I am unable to reproduce the error. So, reverted to lint-staged.

@avdeev avdeev merged commit e7e0602 into avdeev:master Jun 21, 2018
@avdeev
Copy link
Owner

avdeev commented Jun 21, 2018

@prateekrastogi Thanks! I released it.

@prateekrastogi
Copy link
Contributor Author

@avdeev Thanks. Just added the dependency in my project 😁

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.

3 participants