|
| 1 | +# Contribution |
| 2 | + |
| 3 | +Thank you for contributing to `purescript-parsing`! |
| 4 | + |
| 5 | +Following these guidelines helps ensure we're able to carefully consider your contribution and help you finalize your pull request. Your time (like ours) is valuable, and we try to at least provide constructive feedback on every contribution. |
| 6 | + |
| 7 | +### Contributions we love |
| 8 | +`purescript-parsing` is an open-source library and we love to receive contributions. There are many ways you can contribute: |
| 9 | + |
| 10 | +* Help expand our test coverage |
| 11 | +* Help fix open issues, even if that simply means adding a helpful comment |
| 12 | +* Help push along open pull requests that need more work to be complete |
| 13 | +* Improve or add new usage examples to better demonstrate how to use the library in real-world scenarios |
| 14 | +* Update documentation and tutorials to be easier to understand, more comprehensive, and above all -- up to date! |
| 15 | + |
| 16 | +### Contributions we should discuss |
| 17 | +Some contributions will take some discussion before we accept an update to the project. If your contribution includes one of the below examples (or seems to be reasonably similar), please consider reaching out to us before putting in a lot of work to build the feature. We're active on the [PureScript user forum](https://purescript-users.ml) and the [functional programming Slack](https://functionalprogramming.slack.com/) community (new? [use this link to join](https://fpchat-invite.herokuapp.com/)!). For feature requests, feel free to open an issue with a tag. |
| 18 | + |
| 19 | +* New features and functions that will change the library's public API |
| 20 | +* New tutorials (while we love seeing new tutorials, we won't always be able to feature them in the readme) |
| 21 | + |
| 22 | +# Ground Rules |
| 23 | + |
| 24 | +We have a small set of quality-of-life guidelines for contributing to `purescript-parsing`. These include: |
| 25 | + |
| 26 | +* All pull requests must pass continuous integration. |
| 27 | +* If you are adding new functionality, you should provide tests and documentation for your code. If you're fixing an existing bug, please provide a failing test case your patch solves. |
| 28 | +* If possible, please avoid requiring new dependencies. |
| 29 | + |
| 30 | +### Filing issues |
| 31 | +If you have a general question about the project, it's best to ask on the [PureScript user forum](https://purescript-users.ml) or the [functional programming Slack](https://functionalprogramming.slack.com/) than to open a new issue. If you have run into a bug in the project, then please do open an issue! When you do, we ask that you follow a few steps which are outlined in our issues template. The gist of it is here: |
| 32 | + |
| 33 | +* Verify the problem is indeed with `purescript-parsing` (not with Pulp, Bower, or PureScript); |
| 34 | +* Record what versions you are using for PureScript and `purescript-parsing` |
| 35 | +* Describe the issue with steps to reproduce (as much as you are able). A minimal reproducible example is the absolute best case scenario. |
| 36 | + |
| 37 | +We promise to address the issue as soon as we can. |
| 38 | + |
| 39 | +### Suggesting features or enhancements |
| 40 | +We love to hear about ways we could make better. If you're wishing for a feature that doesn't exist in `purescript-parsing`, you're probably not alone; there are bound to be others with similar needs. Please feel free to open an issue on GitHub that describes: |
| 41 | + |
| 42 | +* the feature you would like to see |
| 43 | +* why you need it |
| 44 | +* how it should work |
| 45 | + |
| 46 | +We promise to review your issue, but we aren't always able to accommodate all requests. It helps if you're able to contribute to the implementation, too! |
| 47 | + |
| 48 | +### Code reviews |
| 49 | +The core team looks at pull requests weekly, at which point we will review your code, ensure it meets our ground rules and fits with the philosophy of the project, and -- if necessary -- provide constructive feedback. As soon as at least one member of the team has signed off on your pull request, we will merge your contribution. |
| 50 | + |
| 51 | +If you opened a pull request and we need to ask a few questions, we expect responses within a month. After that we may close the pull request if it isn't showing any activity to keep things manageable. |
0 commit comments