Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 3b8bca6

Browse files
docs: Audit and streamline CONTRIBUTING.md (typescript-eslint#5595)
* docs: Audit and streamline CONTRIBUTING.md * Update CONTRIBUTING.md Co-authored-by: Josh Goldberg <[email protected]> Co-authored-by: Josh Goldberg <[email protected]>
1 parent bb46ef0 commit 3b8bca6

File tree

2 files changed

+32
-8
lines changed

2 files changed

+32
-8
lines changed

.cspell.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,8 @@
115115
"unoptimized",
116116
"unprefixed",
117117
"upsert",
118-
"Zacher"
118+
"Zacher",
119+
"tseslint"
119120
],
120121
"overrides": [
121122
{

CONTRIBUTING.md

Lines changed: 30 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,25 +4,32 @@
44

55
Feel free to raise an issue if you have a question, an enhancement, or a bug report.
66

7-
Use the issue search functionality to search all **_opened and closed_** issues before raising a new issue. If you raise a duplicate issue, you're just creating noise for everyone watching this repo.
7+
Use the issue search functionality to search all **_opened and closed_** [issues](https://github.com/typescript-eslint/typescript-eslint/issues) before raising a new issue. If you raise a duplicate issue, you're just creating noise for everyone watching this repo.
88

99
Before raising a bug, ensure you are using the latest version of our packages. We release every week, so there's a good chance your issue might have already been fixed.
1010

11-
Finally, when raising a new issue, please fill out the issue template - **_please don't skip sections_**.
11+
Finally, when raising a new issue, please fill out the [issue template](https://github.com/typescript-eslint/typescript-eslint/issues/new/choose) - **_please don't skip sections_**.
1212

1313
Please provide **_as much information as possible_**. This project is maintained by volunteers, so the more information you provide, the less likely we will have to waste everyone's time in asking you for more information.
1414

15-
If you have a particularly complex issue - consider creating a small, self-contained reproduction repo. This will help you in figuring out the exact problem, and will help us in reproducing and diagnosing the bug.
15+
If you have a particularly complex issue that can't be reproduced on [our playground](https://typescript-eslint.io/play) - consider creating a small, self-contained reproduction repo. This will help you in figuring out the exact problem, and will help us in reproducing and diagnosing the bug.
1616

1717
**_Help us to help you_**
1818

19+
## Questions and requests for support
20+
21+
Questions and requests for support should not be opened as issues and should be handled in the following ways:
22+
23+
- Ask a question on [StackOverflow](https://stackoverflow.com/questions/tagged/typescript-eslint) using the `typescript-eslint` tag.
24+
- Publicly tweet [@tseslint on Twitter](https://twitter.com/tseslint).
25+
1926
## Commenting
2027

2128
Feel free to comment on any open issue if you have more information that you feel like you can provide. If you don't have more information, instead use the "reaction" feature on the root comment for the issue. We use reactions to help gauge which issues are important to the community, so these are the best way to show us an issue is important.
2229

2330
Please refrain from leaving useless comments on issues. Comments like "+1", or "when's this getting fixed", or "any progress on this" just serve as spam, and annoy every single person subscribed to the issue. Generally we will just delete those comments, so save everyone time and think twice.
2431

25-
Please refrain from commenting on old, closed issues and PRs. Your issue is rarely related enough to a closed issue to warrant "necroing" a dead thread - raising a new issue means you can fill in the template, and make it easier for us to help you. Often times if you comment on a closed issue, we will just ask you to open a new issue, so please save everyone's time, and **_help us to help you_**.
32+
Please refrain from commenting on old, closed issues and PRs. Your issue is rarely related enough to a closed issue to warrant "necroing" a dead thread - raising a new issue means you can fill in the [template](<(https://github.com/typescript-eslint/typescript-eslint/issues/new/choose)>), and make it easier for us to help you. Often times if you comment on a closed issue, we will just ask you to open a new issue, so please save everyone's time, and **_help us to help you_**.
2633

2734
Please refrain from commenting on `main` commits. Commit comments are not searchable, meaning that nobody else can discover your comments. Raise an issue and reference the commit instead so that everyone can see your comment, and you can fill out the template.
2835

@@ -70,18 +77,34 @@ We have a sophisticated CI process setup which gets run on every PR. You must pa
7077
Once your changes are ready, you can raise a PR. The title of your PR should match the following format:
7178

7279
```text
73-
<tag>(<package>): <short description>
80+
<type>(<package>): <short description>
7481
```
7582

76-
Where `<tag>` is one of:
83+
You can find more samples of good past PR titles in [recent commits to `main`](https://github.com/typescript-eslint/typescript-eslint/commits/main))
84+
85+
```text
86+
fix(scope-manager): correct handling for class static blocks
87+
```
88+
89+
```text
90+
docs: Fix links to getting started in README.md
91+
```
92+
93+
### Type
94+
95+
Must be one of the following:
7796

7897
- `feat` - for any new functionality additions
7998
- `fix` - for any bug fixes that don't add new functionality
8099
- `test` - if you only change tests, and not shipped code
81100
- `docs` - if you only change documentation, and not shipped code
82101
- `chore` - anything else
83102

84-
And `<package>` is the name of the package you have made changes within (`eslint-plugin`, `parser`, `typescript-estree`, etc). If you make significant changes across multiple packages, you can omit this (i.e. `feat: foo bar`).
103+
### package
104+
105+
`<package>` is the name of the package you have made changes within (`eslint-plugin`, `parser`, `typescript-estree`, etc). If you make significant changes across multiple packages, you can omit this (i.e. `feat: foo bar`).
106+
107+
### short description
85108

86109
And `<short description>` is a succinct title for the PR.
87110

0 commit comments

Comments
 (0)