Skip to content
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

Incorrect comments in some public member documentation #1109

Open
lukebakken opened this issue Nov 15, 2021 · 4 comments
Open

Incorrect comments in some public member documentation #1109

lukebakken opened this issue Nov 15, 2021 · 4 comments
Assignees
Milestone

Comments

@lukebakken
Copy link
Contributor

User @domicioam noticed the following incorrect comments in #1105:

141652977-26c68741-38e0-4c48-b5fb-32cf4ab3bdb4

@lukebakken lukebakken self-assigned this Nov 15, 2021
@lukebakken
Copy link
Contributor Author

@domicioam please understand that when we suggest opening a pull request to fix issues it is to confirm that yes, the issue is valid and that yes, we would appreciate a contribution to fix it.

@dominsights
Copy link

dominsights commented Nov 15, 2021

@domicioam please understand that when we suggest opening a pull request to fix issues it is to confirm that yes, the issue is valid and that yes, we would appreciate a contribution to fix it.

Hello Luke,

I am not sure if that was the intention. I felt like the other guy was calling me lazy for reporting it instead of fixing it. I would gladly fix the comments if I was familiar with the library, but the best I could do at the time was to report the issue.

Thanks for the heads up anyways. It is good to learn the jargon of open source projects.

Kind regards,
Dom.

@lukebakken
Copy link
Contributor Author

I am not sure if that was the intention. I felt like the other guy was calling me lazy for reporting it instead of fixing it.

I am certain that @michaelklishin's intention is to suggest contributing to the library, not suggest anyone is lazy. The RabbitMQ core team (I'm part of it) receives many issue reports across a large number of repositories here on GitHub. A fraction of those (like yours) are legitimate and we suggest opening pull requests in case people are hesitant to do so. If an issue doesn't require a fix we say so as well.

In general, open-source project maintainers may seem abrupt with their comments but it's best to take them at face value rather than reading into them. We're all busy with our day jobs!

@michaelklishin
Copy link
Member

@domicioam I did not imply that anyone is lazy. It's a perfectly common thing to suggest for a maintainer: if the problem is well understood, the reporter might as well submit a PR.

@lukebakken lukebakken added this to the 7.0.0 milestone Nov 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants