Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Commit

Permalink
Merge remote-tracking branch 'joshtriplett/responding' into main
Browse files Browse the repository at this point in the history
  • Loading branch information
nikomatsakis committed Sep 8, 2021
2 parents f6827a1 + 11208fc commit 57d1d52
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/how_to_rustacean/bring_joy.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ Somebody tweets about a terrible error message. It has a type that is 16K long a

Someone opens an issue claiming a bug in Rust. In fact, the problem is that their code was relying on an implementation detail of a method in the standard library -- this is despite the fact that the documentation for the function clearly states that this detail may change.

**Just right:** You thank them for higlighting this change, and point out that the problem is actually not a bug in Rust, but rather a change that is within the documentation.
**Just right:** You thank them for higlighting this change, and point out that the problem is actually not a bug in Rust, but rather a change that is within the documentation. If appropriate, you might express sympathy for the impact of this change on them, and you consider if there *may* be a way to better advertise or guard against such impact in the future, but you close the current issue.

**Too much:** You apologize profusely and revert the PR that made the change. You wind up in a long conversation with the

**Too little:** You reply curtly, "Not a bug, RTFM", and close the issue.
**Too little:** You reply curtly, "Not a bug, RTFM", and close the issue.

0 comments on commit 57d1d52

Please sign in to comment.