|
| 1 | +We would love to have you as a contributor to NTVS! |
| 2 | + |
| 3 | +## Contributing Issues |
| 4 | +When creating new issues in the issue tracker, please follow these guidelines: |
| 5 | + |
| 6 | +- Use a descriptive title that identifies the issue to be addressed or the requested feature (e.g., "Feature F should report ABC when XYZ is used in DEF"). |
| 7 | +- Provide a detailed description of the issue or request feature. |
| 8 | +- For bug reports, please also: |
| 9 | + - Describe the expected behavior and the actual behavior. |
| 10 | + - Provide example code or steps that reproduce the issue. |
| 11 | + - Specify any relevant exception messages and stack traces. |
| 12 | +- Subscribe to notifications for the created issue in case there are any follow-up questions. |
| 13 | + |
| 14 | + |
| 15 | +## Submitting a Pull Request |
| 16 | +All pull requests are reviewed by one of the project maintainers before being merged in. Here are some guidelines that to help the process go as quickly as possible: |
| 17 | + |
| 18 | +- Clearly state what the pull request accomplishes. |
| 19 | + - If it's a bug fix, what was the bug and how has it been fixed? |
| 20 | + - If you're adding a new feature, what is it and what does it do for users? |
| 21 | +- Provide a high level explanation of the code changes. This will help reviewers understand your implementation choices and will speed up the review process. |
| 22 | + - Please try to keep pull requests small and focused. |
| 23 | +- Make sure that existing tests continue to pass. |
| 24 | +- All new code must include tests. |
| 25 | + - For a bug fix, consider adding a test that would fail before your change, but passes now. |
| 26 | + - Feel free to ask for advice if you are unsure how what to test or how to integrate into our test infrastructure. |
| 27 | +- There is no formal code style-guide, but please try to match the style of any file you are editing. |
| 28 | + |
| 29 | +If you are unsure of anything, feel free to ask a question. We're always happy to help. |
| 30 | + |
| 31 | +### Licensing and Legal |
| 32 | +Node.js Tools for Visual Studio is a member of [Microsoft](http://microsoft.github.io/) and operates under the [Apache 2.0 license](https://github.com/Microsoft/nodejstools/blob/master/LICENSE). |
| 33 | + |
| 34 | +> ❗ ️️**Important:** You must sign the [Microsoft Contributor License Agreement (CLA)][cla] before submitting your pull-request. This only takes a minute or two and can be completed entirely online. All you need to do is [submit a request via an online form][cla] and electronically sign the CLA when you receive the email with document link. The CLA only needs to be completed once, and covers all Microsoft open source software projects. |
| 35 | +
|
| 36 | +All contributions will be made under the Apache 2.0 license. Please don’t submit anything with any other licensing statements. If you want to make a contribution that includes code that under a different license, please let us know so we can figure out what to do. Also, make sure you have the right to make your contribution, including clearance from your employer if applicable. |
| 37 | + |
| 38 | + |
| 39 | +[cla]: https://cla.microsoft.com/ |
0 commit comments