-
-
Notifications
You must be signed in to change notification settings - Fork 612
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
Create CONTRIBUTING.md #3574
Comments
From @ksuess in #3934 (comment):
|
@stevepiercy I think a lot of this was addressed by plone/documentation#1478 and #4634. Is there any part left to do or can we close the issue? |
I updated the checklist. There is still some work to perform. |
Actually this issue is a mess. I have a better idea. Closing in favor of #4665 |
Is your feature request related to a problem? Please describe.
New users frequently ask how to contribute to Volto, and I do not know the authoritative location for that.
I think we use the following:
The last two are way confusing. 😕 See screenshot as rendered in Documentation:
The standard is to have a
CONTRIBUTING.md
file at the root of a project. GitHub uses a repo'sCONTRIBUTING.md
when new users open an issue or pull request for the first time, and this would help guide them to be more successful. This does not happen currently in Volto because we lack aCONTRIBUTING.md
.Describe the solution you'd like
We have a default CONTRIBUTING.md for the Plone organization, but that is insufficient.
We should have a local
CONTRIBUTING.md
in the repo that overrides the organization level file. That file should be placed in the root of the project (that's where developers look first), or alternatively in.github/
.We should then add a file in the Volto documentation that includes the new
CONTRIBUTING.md
.Finally, we would need to update the Makefile in Documentation to clone the root of the Volto repo, instead of just its documentation, so that we can properly pull in the file and build the entire documentation.
I would appreciate your thoughts and feedback on this.
The text was updated successfully, but these errors were encountered: