-
I have finded the problem and suggest a way to fix that. and submited a PR for that, but unlike the general merge system, I am a little confused by the fact that the problem is solved outside of my PR with a only simple (i.e., not including me) comment in other PR. Am I missing something? I think this kind of non-recognition of even small contributions can be demoralizing to contributors. Is there any way we can ensure small contributions are better recognized in the future? |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 1 reply
-
Apologies @elecbug. I'll call out your contribution in the release notes for the next version in which we'll release this fix. In general though, I wouldn't call out a bug report by itself unless it's substantial. If you're looking to contribute, may I humbly suggest helping review PRs? I'd acknowledge the contribution as a Git commit footer message at least (something like reviewed-by: foo). In the case of pushing a new PR, at minimum it should include tests. It should include a motivation for the change. And it should be mindful of future maintenance efforts. It's better to start discussion on an issue, and then create a PR. Again, no slight intended here. Just trying to make a great and useful library. Thanks for understanding. |
Beta Was this translation helpful? Give feedback.
Apologies @elecbug. I'll call out your contribution in the release notes for the next version in which we'll release this fix.
In general though, I wouldn't call out a bug report by itself unless it's substantial. If you're looking to contribute, may I humbly suggest helping review PRs? I'd acknowledge the contribution as a Git commit footer message at least (something like reviewed-by: foo).
In the case of pushing a new PR, at minimum it should include tests. It should include a motivation for the change. And it should be mindful of future maintenance efforts. It's better to start discussion on an issue, and then create a PR.
Again, no slight intended here. Just trying to make a great an…