-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Note about fix rules in docs #4774
Comments
@SpainTrain Thanks for the issue! If you're reporting a bug, please be sure to include:
Requesting a new rule? Please see Proposing a New Rule for instructions. |
Can you explain what you have in mind? (We really need more than a short phrase in issue descriptions to decide if we want agree or not.) |
Apologies, this was just created because I wanted to open a PR and a "fixes" issue is required. I will be more descriptive in these in the future even if it is just for a PR. |
Thanks! What we'd like is for an issue that describes a problem, and then submit a PR that has the solution. That way, if we don't like the solution, we can close the PR and try another one, all the while the issue remains open as a clear description of the problem. |
@nzakas I know this wasn't accepted yet but I wanted to give this one-sentence patch a shot to see if it'd address @SpainTrain's concern. |
Add note about fix rules to docs
Related #4766 #4767
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: