Improve contributing documentation clarity#2568
Improve contributing documentation clarity#2568Karanrajpurohit855 wants to merge 2 commits intoarviz-devs:mainfrom
Conversation
aloctavodia
left a comment
There was a problem hiding this comment.
Thanks for your contribution. This may be potentially useful, but it needs much more work.
|
|
||
| If you are new to open source: | ||
| - Start with small documentation improvements | ||
| - Look for issues labeled "User Documentation" |
There was a problem hiding this comment.
We don't have this label. I think the label is "documentation", please double-check. Also, "good first issue" could be useful (also double check the name).
There was a problem hiding this comment.
I see that we have that label, my bad!
There was a problem hiding this comment.
Moreover, I strongly disagree with this statement. Things labeled "User Documentation" are not necessarily easier than the rest. If a tag is mentioned it should be the "Beginner" one but we don't have much capacity to dedicate to issue triaging, so I would instead focus on advise that is not label dependent. It is key to understand what the issue is talking about, if it is still relevant, if it can be reproduced... Even for issues labelled "Beginner" when we have them they might be for someone who is a beginner on open source but already an intermediate ArviZ user, or for someone who is new to ArviZ but somewhat familiar with open source.
| We welcome a wide range of contributions, not only code! | ||
| Even improving documentation or fixing typos is a valuable contribution to ArviZ. | ||
|
|
||
| ## How to contribute |
There was a problem hiding this comment.
This is only for people that wants to contribute code. You don't need to do this to report issues. So it should be in a different place, maybe a subsection of "contributing code". Additionally, if someone is a beginner, does this list help or do we need something more explicit? And if not a beginner, not very useful, which makes the "For Beginners" heading weird.
There was a problem hiding this comment.
I think this whole "how to contribute" section should be removed. This is only a bullet point version of the content already present in the first link within the "Contributing code" section of this same page.
OriolAbril
left a comment
There was a problem hiding this comment.
The PR description says this is supposed to close #1754 but at least for now this is very clearly not the case. The issue is very clear that the updates should be done to the contributing guide on the website, not to this CONTRIBUTING.md and has a much larger scope
| 5. Commit and push | ||
| 6. Open a Pull Request | ||
|
|
||
| ## For Beginners |
There was a problem hiding this comment.
It might be useful to add some content specific to how beginners can find things to work on within the contributing guide. That being said, I am strongly against adding this directly to this markdown file. We already have a much more extensive contributing guide on the website so I think we should keep this page extremely short and with links to that page. A potential location for this kind of content could be https://python.arviz.org/en/stable/contributing/contributing_prs.html
Fixes #1754
Improved contributing documentation by adding clearer steps and beginner-friendly instructions.