Skip to content

Travis-CI & Coveralls support #39

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Travis-CI & Coveralls support #39

wants to merge 11 commits into from

Conversation

vinayakkulkarni
Copy link
Contributor

  • Coveralls support added
  • Travis-CI support added
  • Updated dependencies to latest version - Fixes npm audit vulnerabilities.
  • Added badges to README.md

@jackmellis : These are the following steps you need to do:

  1. goto Travis CI and enable this repo
  2. goto Coveralls and enable this repo
  3. release a newer version on npm & github

Only then you'd be able to see the badges like this:
Screenshot 2019-04-15 at 7 27 02 PM

I'll close #38 if you're able to merge this @jackmellis

@vinayakkulkarni
Copy link
Contributor Author

FYI:
Build result -> https://travis-ci.org/vinayakkulkarni/require-extension-hooks-vue/builds/532257160

@vinayakkulkarni vinayakkulkarni mentioned this pull request May 14, 2019
Closed
@vinayakkulkarni
Copy link
Contributor Author

@jackmellis 🛎

@vinayakkulkarni
Copy link
Contributor Author

Bump

@vinayakkulkarni
Copy link
Contributor Author

Bump

2 similar comments
@vinayakkulkarni
Copy link
Contributor Author

Bump

@vinayakkulkarni
Copy link
Contributor Author

Bump

@vinayakkulkarni
Copy link
Contributor Author

bump

@vinayakkulkarni
Copy link
Contributor Author

Buuuuuump!

@vinayakkulkarni
Copy link
Contributor Author

🛎

@vinayakkulkarni
Copy link
Contributor Author

bump

@vinayakkulkarni
Copy link
Contributor Author

bummppppp

@vinayakkulkarni
Copy link
Contributor Author

bump

@vinayakkulkarni
Copy link
Contributor Author

@jackmellis mate any chance you wanna merge this?

fixes lodash vulnerabilities too

@vinayakkulkarni
Copy link
Contributor Author

Hey @jackmellis, any updates?

@vinayakkulkarni
Copy link
Contributor Author

Bump 🛎

@vinayakkulkarni
Copy link
Contributor Author

Bump

1 similar comment
@vinayakkulkarni
Copy link
Contributor Author

Bump

@vinayakkulkarni
Copy link
Contributor Author

BUMMMMMP!

@vinayakkulkarni
Copy link
Contributor Author

Bump :)

@vinayakkulkarni
Copy link
Contributor Author

Bump

@vinayakkulkarni
Copy link
Contributor Author

Bump 🛎

2 similar comments
@vinayakkulkarni
Copy link
Contributor Author

Bump 🛎

@vinayakkulkarni
Copy link
Contributor Author

Bump 🛎

@vinayakkulkarni
Copy link
Contributor Author

bump 🛎

@vinayakkulkarni
Copy link
Contributor Author

@jackmellis Any update on this?!

@vinayakkulkarni
Copy link
Contributor Author

🛎

1 similar comment
@vinayakkulkarni
Copy link
Contributor Author

🛎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vinayakkulkarni