This guide is designed to help you get off the ground quickly contributing to Trails.js, Trailpacks and the Trails.js ecosystem. The goal of our community is to make it easy for members of all skill levels to contribute. This guide will help you write useful issues, propose eloquent feature requests, and submit top-notch code that can be merged quickly.
Maintaining a open source project is a labor of love, meaning the core maintainers of Trails.js or Trailpacks are volunteering their time. Respecting the guidelines laid out below helps the maintainers be efficient and make the most of the time they spend working on the project. This, in turn, creates a better experience of working with Trails more enjoyable for the community at large.
Trails is composed of a core library, Trails.js, and a number of Trailpacks, which have their own dedicated repositories. These repositories may also live outside the Trails.js official Github organization.
Please open issues with Waterline, various trailpacks, various generators, etc. in the relevant repo.
This helps us stay on top of issues and keep organized.
When submitting an issue, please follow these simple instructions:
- Search for issues similar to yours in GitHub search and Google.
- Feature requests are welcome; see Requesting Features below for submission guidelines.
- If there's an open issue, please contribute to that issue.
- If there's a closed issue, open a new issue and link the url of the already closed issue(s).
- If there is no issue, open a new issue and specify the following:
- A short description of your issue in the title
- The trails version (find this with in the package.json file)
- Detailed explanation of how to recreate the issue, including necessary setup setps
- If you are experiencing more than one problem, create a separate issue for each one. If you think they might be related, please reference the other issues you've created.
New feature requests should be made as pull requests to the
backlog
section of ROADMAP.MD or as issues on theBacklog
milestone in the issue queue. We will monitor community discussion on these PRs and issues and if they are wanted by the community/trails devs, they will be merged. Further discussion is welcome even after a PR has been merged.
- First, look at the
backlog
table in ROADMAP.MD or the Backlog Milestone in the issue queue toand also search open pull requests in that file to make sure your change hasn't already been proposed. If it has, join the discussion. - If it doesn't already exist, create a pull request editing the
backlog
table of ROADMAP.MD. - Start a discussion about why your feature should be built (or better yet, build it). Get feedback in the Trails.js Gitter Channel. The more feedback we get from our community, the better we are able to build the framework of your dreams 🌲
Ideally, all code contributions should be accompanied by functional and/or unit tests (as appropriate).
Test Coverage:
Edge (master branch) |
---|
The community is what makes Trails great, without you we wouldn't have come so far. But to help us keep our sanity and reach code-nirvana together, please follow these quick rules whenever contributing.
Note: This section is based on the Node.js contribution guide.
For consistency, all code in Trails core, including core hooks and core generators, must be written in JavaScript, not CoffeeScript or TypeScript. We can't merge a pull request in CoffeeScript.
If the Trailpack is in the Trails Github organization, please send feature requests, patches and pull requests to that organization. Other Trailpacks may have their own contribution guidelines. Please follow the guidelines of the Trailpack you are contributing to.
You are welcome to author a new Trailpack at any time. Trailpacks must inherit from the main Trailpack interface to inherit the API. Feel free to start work on a new trailpack, just make sure and do a thorough search on npm, Google and Github to make sure someone else hasn't already started working on the same thing.
It is recommended that you maintain your Trailpack in your own Github repository. If you would like to submit your Trailpack to be listed in the Trails.js Github Organization, please submit an issue to the Trails Issue queue.
Trails generators are based upon Yeoman. You can read up on Yeoman's authoring guide for how to write your own generator or contribute to another generator. Please follow the core best practices for contributing to generators. If it is located in a different repo, please send feature requests, patches, and issues there.
Trails has several dependencies referenced in the package.json
file that are not part of the project proper. Any proposed changes to those dependencies or their dependencies should be sent to their respective projects (i.e. Waterline etc.) Please do not send your patch or feature request to this repository, we cannot accept or fulfill it.
In case of doubt, open an issue in the issue tracker, ask your question in the Gitter room. Especially if you plan to work on something big. Nothing is more frustrating than seeing your hard work go to waste because your vision does not align with a project's roadmap. At the end of the day, we just want to be able to merge your code.
- If you don't know how to fork and PR, Github has some great documentation. Here's the quick version:
- Fork the repo.
- Add a test for your change. Only refactoring and documentation changes require no new tests. If you are adding functionality or fixing a bug, we need a test!
- Make the tests pass and make sure you follow our syntax guidelines.
- Add a line of what you did to CHANGELOG.md (right under
master
). - Push to your fork and submit a pull request to the appropriate branch
- Trails is sponsored by Balderdash, a Node.js development studio in Norfolk, VA. If you have a mutually beneficial opportunity to work together, or want to fund us to accelerate the development of a feature in Trails for a real-world use case, please contact us.
- Donate to help me (@tjwebbusa) stop paying myself, and instead use that money to expand the core team, improve docs, etc.