|
1 | 1 | # Contributing |
2 | 2 |
|
3 | | -Thank you for your interest in contributing to TypeScript ESLint! 💜 |
4 | | - |
5 | | -> Make sure you read our [Code of Conduct](./CODE_OF_CONDUCT.md) before contributing. |
6 | | -
|
7 | | -## Raising Issues |
8 | | - |
9 | | -So you've got a bug report, documentation request, or feature suggestion? |
10 | | -Great! |
11 | | - |
12 | | -Do: |
13 | | - |
14 | | -- Make sure you're using the [latest version of our packages](https://github.com/typescript-eslint/typescript-eslint/releases) |
15 | | -- Search [all opened and closed issues](https://github.com/typescript-eslint/typescript-eslint/issues?q=is%3Aissue+) to make sure your issue wouldn't be a duplicate |
16 | | -- Fill out the [appropriate issue template](https://github.com/typescript-eslint/typescript-eslint/issues/new/choose) completely |
17 | | -- Provide as much information as you can |
18 | | - |
19 | | -### Questions and Support Requests |
20 | | - |
21 | | -We do not have the bandwidth to handle questions or support requests in the issue tracker. |
22 | | -You can instead: |
23 | | - |
24 | | -- Ask a question on [StackOverflow](https://stackoverflow.com/questions/tagged/typescript-eslint 'StackOverflow questions tagged with typescript-eslint') using the `typescript-eslint` tag |
25 | | -- Publicly tweet [@tseslint on Twitter](https://twitter.com/tseslint) |
26 | | - |
27 | | -> Note that requests to add documentation _are_ allowed, even encouraged! 📝 |
28 | | -
|
29 | | -## Commenting |
30 | | - |
31 | | -Please do comment on any open issue if you have more information that would be useful. |
32 | | - |
33 | | -Don't: |
34 | | - |
35 | | -- Leave useless comments such as _"+1"_ or _"when's this getting fixed?"_ that only act as spam |
36 | | - - If you have nothing to add but enthusiasm and joy, add a reaction such as 👍 |
37 | | -- Bring up unrelated topics in existing issues: instead, file a new issue |
38 | | -- Comment on closed PRs: instead, [file a new issue](#raising-issues) |
39 | | -- Comment on commits directly, as those comments are not searchable: instead, file a new issue |
40 | | - |
41 | | -## Pull Requests |
42 | | - |
43 | | -> See [DEVELOPMENT.md](./DEVELOPMENT.md) for details on how to get started developing locally. |
44 | | -
|
45 | | -Do: |
46 | | - |
47 | | -- Only send pull requests that resolve [open issues marked as `accepting prs`](https://github.com/typescript-eslint/typescript-eslint/issues?q=is%3Aissue+is%3Aopen+label%3A%22accepting+prs%22) |
48 | | - - One exception: extremely minor documentation typos |
49 | | -- Fill out the pull request template in full |
50 | | -- Validate your changes per [Development > Validating Changes](./DEVELOPMENT.md#validating-changes) before un-[drafting your PR](https://github.blog/2019-02-14-introducing-draft-pull-requests) |
51 | | - |
52 | | -Don't: |
53 | | - |
54 | | -- Force push after opening a PR |
55 | | - - Reasoning: GitHub is not able to track changes across force pushes, which makes it take longer for us to perform incremental reviews |
56 | | -- Comment asking for updates |
57 | | - - Reasoning: Your PR hasn't been forgotten! The volunteer maintainers have limited time to work on the project, and they will get to it as soon as they are able. |
58 | | - |
59 | | -### Raising a PR |
60 | | - |
61 | | -Once your changes are ready, you can raise a PR! 🙌 |
62 | | -The title of your PR should match the following format: |
63 | | - |
64 | | -```text |
65 | | -<type>(<package>): <short description> |
66 | | -``` |
67 | | - |
68 | | -You can find more samples of good past PR titles in [recent commits to `main`](https://github.com/typescript-eslint/typescript-eslint/commits/main). |
69 | | - |
70 | | -```text |
71 | | -fix(scope-manager): correct handling for class static blocks |
72 | | -``` |
73 | | - |
74 | | -```text |
75 | | -docs: Fix links to getting started in README.md |
76 | | -``` |
77 | | - |
78 | | -Within the body of your PR, make sure you reference the issue that you have worked on, as well as pointing out anything of note you wish us to look at during our review. |
79 | | - |
80 | | -> We do not care about the number, or style of commits in your history, because we squash merge every PR into `main`. |
81 | | -> Feel free to commit in whatever style you feel comfortable with. |
82 | | -
|
83 | | -#### type |
84 | | - |
85 | | -Must be one of the following: |
86 | | - |
87 | | -<!-- Keep this synchronized with /.github/workflows/semantic-pr-titles.yml --> |
88 | | - |
89 | | -- `docs` - if you only change documentation, and not shipped code |
90 | | -- `feat` - for any new functionality additions |
91 | | -- `fix` - for any bug fixes that don't add new functionality |
92 | | -- `test` - if you only change tests, and not shipped code |
93 | | -- `chore` - anything else |
94 | | - |
95 | | -#### package |
96 | | - |
97 | | -The name of the package you have made changes within, (e.g. `eslint-plugin`, `parser`, `typescript-estree`). |
98 | | -If you make significant changes across multiple packages, you can omit this (e.g. |
99 | | -`feat: foo bar`). |
100 | | - |
101 | | -#### short description |
102 | | - |
103 | | -A succinct title for the PR. |
104 | | - |
105 | | -### Addressing Feedback and Beyond |
106 | | - |
107 | | -With your PR raised and the CI passing, your PR will [wait in the queue to be reviewed](https://github.com/typescript-eslint/typescript-eslint/pulls?q=is%3Apr+is%3Aopen+sort%3Acreated-asc+-label%3A%22breaking+change%22+-label%3A%22awaiting+response%22+-label%3A%221+approval%22+-label%3A%22DO+NOT+MERGE%22+status%3Asuccess). |
108 | | -We generally review PRs oldest to newest, unless we consider a newer PR higher priority (e.g. if it's a bug fix). |
109 | | - |
110 | | -Once we have reviewed your PR, we will provide any feedback that needs addressing. |
111 | | -If you feel a requested change is wrong, don't be afraid to discuss with us in the comments. |
112 | | -Once the feedback is addressed, and the PR is reviewed, we'll ensure the branch is up to date with `main`, and merge it for you. |
| 3 | +See **https://typescript-eslint.io/contributing** for our contributing guidelines. |
| 4 | +Thanks! 💖 |
0 commit comments