For support or usage questions like "How do I do X with vue-directive-tooltip." and "My code doesn't work.", please search and ask on StackOverflow with a 'vue-directive-tooltip' tag first.
The ideal GitHub issue (and even some feature requests) is not an issue, it's a PR with a failing test case.
@rauchg
Before filing an issue please search the issue tracker; your issue may have already been discussed or fixed in master
.
If you want your issue to get priority, submit it as a PR instead
Fill in the template provided by the issue tracker, try to be as clear and complete as possible, providing a Short, Self Contained, Correct (Compilable), Example (SSCE) (link to a repository) helps a lot .
Feature or enhancement requests should be submitted in the issue tracker, with a description (follow the template) of the expected behavior & use case, where they’ll remain until approval by the lead maintainer(s) and/or enough interest from the community.
You can request a feature by writing a pull request, but this is no guarantee it will be merged.
There are issues marked with the 'help wanted' label.
This is a perfect start if you want to help out with the further development of vue-directive-tooltip.
In general, the contribution workflow looks like this:
- Fork the repo.
- Clone the repo.
git clone https://github.com/your-username/vue-directive-tooltip.git
. - Create a new branch based off the master branch, provide a descriptive name
(ex. 'feat-add-better-logging', 'bug-removed-double-method', 'enh-bumped-eslint') - Before running the code you’ll need to install the dependencies (
npm install
oryarn
). - Implement your feature / bugfix (using the watch scripts), you should only need to modify
/src
. Don’t worry about regenerating the build folders (/cjs
,/es
,/dist
), they are built in the prepublish phase. - Make sure all tests pass, coverage is 100% and there are no linting errors.
- Submit a PR, referencing what it addresses.
- Please try to keep your PR focused in scope and avoid including unrelated commits.
After you have submitted your PR, we'll try to get back to you as soon as possible.
We will review your PR and might request changes.
You can run the watch scripts
npm run build:watch
and
npm run test:watch
to make development easier.
Please follow the conventions already established in the code.
Guidelines are enforced using ESLint.
You can run the linting script by using
$ npm run lint
Include updated (or add new) tests in the __tests__
directory as part of your PR.
You can run the test script by using
$ npm run test
or with coverage
$ npm run test:coverage
Aim for 100% coverage.
We added pre-commit and pre-push hooks to make sure you don't commit non-working code.