You should make a pull request if you want:
- to fix bug for existing functions
- to enhance the API or implementation of an existing function
- to add a function that is a representation of a class in
java.util
- to implement an issue
In the case of adding a new function, that function must follow the Definition of Done.
A previous discussion in issues section is preferred before working on a PR, to avoid wasting of time working on something useless.
- no build issue
- no lint issue
- at least one spec file per unit
- all tests green
- full code coverage
- full public API documented
- at least one demo scenario per unit
First of all, this is a TypeScript project that's distributed on npmjs.org and therefore uses JavaScript tooling based on Node.js with dependencies from npm. You're going to need to have those things installed to contribute to this project.
- Fork the repo
- Clone your fork
- Create a branch
- Run
npm install
- Run
npm run test:single && npm run build
. If everything works, then you're ready to make changes. - If you want, you can run
npm test
and see that it's watching your file system for changes. - Run
npm run docs
and generate documentation for your changes, especially if your changes affected functions signature. - If you get things working, add your changed files with
git add
and runnpm run commit
to get an interactive prompt for creating a commit message that follows our standards. - Push your changes to your fork with
git push
- Create a pull request.
- Get merged! 🎉 🎊