|
| 1 | +# Contributing to Jet |
| 2 | + |
| 3 | +Thank you for taking the time to contribute to Jet! |
| 4 | +:airplane: :confetti_ball: :tada: :fireworks: :balloon: |
| 5 | + |
| 6 | +Jet is a collaborative effort with the quantum computation community - while we |
| 7 | +will continue working on adding new and exciting features to Jet, we invite you |
| 8 | +to join us and suggest improvements, research ideas, or even just to discuss how |
| 9 | +Jet fits into your workflow. |
| 10 | + |
| 11 | +## How can I get involved in the community? |
| 12 | + |
| 13 | +If you want to contribute but don't know where to start, check out our |
| 14 | +[documentation](https://quantum-jet.readthedocs.io) and have a go at working |
| 15 | +through some of the tutorials. Afterwards, take a look at the Jet API to see |
| 16 | +how things work under the hood. |
| 17 | + |
| 18 | +## How can I contribute? |
| 19 | + |
| 20 | +It's up to you! |
| 21 | + |
| 22 | +* **Be a part of our community** - provide exciting updates of the projects or |
| 23 | + experiments you are investigating with Jet. |
| 24 | + |
| 25 | + You can even write your own Jet tutorials, or blog about your simulation results. |
| 26 | + Send us the link, and we may even add it to our documentation as an external |
| 27 | + resource! |
| 28 | + |
| 29 | +* **Test the cutting-edge Jet releases** - clone our GitHub repository, and keep |
| 30 | + up to date with the latest features. If you run into any bugs, make a bug |
| 31 | + report on our [issue tracker](https://github.com/XanaduAI/jet/issues). |
| 32 | + |
| 33 | +* **Report bugs** - even if you are not using the development branch of Jet, if |
| 34 | + you come across any bugs or issues, make a bug report. See the next section |
| 35 | + for more details on the bug reporting procedure. |
| 36 | + |
| 37 | +* **Suggest new features and enhancements** - use the GitHub issue tracker and |
| 38 | + let us know what will make Jet even better for your workflow. |
| 39 | + |
| 40 | +* **Contribute to our documentation, or to Jet directly** - we are hoping for |
| 41 | + our documentation to become an online, open-source resource for all things |
| 42 | + tensor. If you would like to add to it, or suggest improvements/changes, let |
| 43 | + us know - or even submit a pull request directly. All authors who have |
| 44 | + contributed to the Jet codebase will be listed alongside the latest release. |
| 45 | + |
| 46 | +* **Build an application on top of Jet** - have an idea for an application, and |
| 47 | + Jet provides the perfect computational backbone? Consider making a fully |
| 48 | + separate app that builds upon Jet as a base. Ask us if you have any questions, |
| 49 | + and send a link to your application to [email protected] so we can highlight |
| 50 | + it in our documentation! |
| 51 | + |
| 52 | +Appetite whetted? Keep reading below for all the nitty-gritty on reporting bugs, |
| 53 | +contributing to the documentation, and submitting pull requests. |
| 54 | + |
| 55 | +## Reporting bugs |
| 56 | + |
| 57 | +We use the [GitHub issue tracker](https://github.com/XanaduAI/jet/issues) to |
| 58 | +keep track of all reported bugs and issues. If you find a bug, or have an issue |
| 59 | +with Jet, please submit a bug report! User reports help us make Jet better on |
| 60 | +all fronts. |
| 61 | + |
| 62 | +To submit a bug report, please work your way through the following checklist: |
| 63 | + |
| 64 | +* **Search the issue tracker to make sure the bug wasn't previously reported**. |
| 65 | + If it was, you can add a comment to expand on the issue and share your |
| 66 | + experience. |
| 67 | + |
| 68 | +* **Fill out the issue template**. If you cannot find an existing issue |
| 69 | + addressing the problem, create a new issue by filling out the |
| 70 | + [bug issue template](./ISSUE_TEMPLATE/BUG.md). This template is added |
| 71 | + automatically to the comment box when you create a new issue. Please try and |
| 72 | + add as many details as possible! |
| 73 | + |
| 74 | +* Try and make your issue as **clear, concise, and descriptive** as possible. |
| 75 | + Include a clear and descriptive title, and include all code snippets and |
| 76 | + commands required to reproduce the problem. If you're not sure what caused the |
| 77 | + issue, describe what you were doing when the issue occurred. |
| 78 | + |
| 79 | +## Suggesting features, document additions, and enhancements |
| 80 | + |
| 81 | +To suggest features and enhancements, use the GitHub tracker. No template is |
| 82 | +required for feature requests and enhancements, but here are a couple of |
| 83 | +suggestions for things to include: |
| 84 | + |
| 85 | +* **Use a clear and descriptive title** |
| 86 | +* **Provide a step-by-step description of the suggested feature**. |
| 87 | + |
| 88 | + - If the feature is related to any theoretical results in quantum computation, |
| 89 | + provide any relevant equations. Alternatively, provide references to |
| 90 | + papers or preprints with the relevant sections and equations noted. |
| 91 | + - If the feature is workflow-related, or related to the use of Jet, explain |
| 92 | + why the enhancement would be useful and where/how you would like to use it. |
| 93 | + |
| 94 | +* **For documentation additions**, point us towards any relevant equations, |
| 95 | + papers, and preprints with the relevant sections and equations noted. Short |
| 96 | + descriptions of its use and importance would also be helpful. |
| 97 | + |
| 98 | +## Pull requests |
| 99 | + |
| 100 | +If you would like to contribute directly to the Jet codebase, simply make a |
| 101 | +fork of the main branch and submit a |
| 102 | +[pull request](https://help.github.com/articles/about-pull-requests). We |
| 103 | +encourage everyone to have a go forking and modifying the Jet source code; |
| 104 | +however, we have a couple of guidelines on pull requests to ensure the main |
| 105 | +branch conforms to existing standards and quality. |
| 106 | + |
| 107 | +### General guidelines |
| 108 | + |
| 109 | +* **Do not make a pull request for minor typos/cosmetic code changes** - create |
| 110 | + an issue instead. |
| 111 | +* **For major features, consider making an independent application** that runs |
| 112 | + on top of Jet, rather than modifying Jet directly. |
| 113 | + |
| 114 | +### Before submitting |
| 115 | + |
| 116 | +Before submitting a pull request, please make sure the following is done: |
| 117 | + |
| 118 | +* **All new features must include a unit test.** If you've fixed a bug or added |
| 119 | + code that should be tested, add a test to the [`test/`](test/) directory! |
| 120 | +* **All new classes, functions, and members must be clearly commented and documented**. |
| 121 | + If you do make documentation changes, make sure that the docs build and render |
| 122 | + correctly by running `make docs`. |
| 123 | +* **Ensure that the test suite passes.** Verify that `make test` passes. |
| 124 | +* **Ensure that the modified code is formatted correctly.** Verify that |
| 125 | + `make format` passes. |
| 126 | + |
| 127 | +### Submitting the pull request |
| 128 | +* When ready, submit your fork as a |
| 129 | + [pull request](https://help.github.com/articles/about-pull-requests) to the |
| 130 | + Jet repository, filling out the |
| 131 | + [pull request template](./PULL_REQUEST_TEMPLATE.md). This template is added |
| 132 | + automatically to the comment box when you create a new PR. |
| 133 | + |
| 134 | +* When describing the pull request, please include as much detail as possible |
| 135 | + regarding the changes made, new features, and performance improvements. If |
| 136 | + including any bug fixes, mention the issue numbers associated with the bugs. |
| 137 | + |
| 138 | +* Once you have submitted the pull request, two things will automatically occur: |
| 139 | + |
| 140 | + - The **test suite** will automatically run on |
| 141 | + [CircleCI](https://app.circleci.com/pipelines/github/XanaduAI/jet) to ensure |
| 142 | + that all tests continue to pass. |
| 143 | + |
| 144 | + - The **formatter** will automatically run on |
| 145 | + [CircleCI](https://app.circleci.com/pipelines/github/XanaduAI/jet) to ensure |
| 146 | + that all the code is properly formatted. |
| 147 | + |
| 148 | + Based on these results, we may ask you to make small changes to your branch |
| 149 | + before merging the pull request into the main branch. Alternatively, you can |
| 150 | + [grant us permission to make changes to your pull request branch](https://help.github.com/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork/). |
| 151 | + |
| 152 | +:airplane: Thank you for contributing to Jet! :airplane: |
| 153 | + |
| 154 | +\- The Jet team |
0 commit comments