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

First Time Contributors Documentation (CONTRIBUTING.md) #1290

Open
KristyWaters opened this issue May 25, 2024 · 2 comments
Open

First Time Contributors Documentation (CONTRIBUTING.md) #1290

KristyWaters opened this issue May 25, 2024 · 2 comments
Assignees

Comments

@KristyWaters
Copy link

I would love it if we could expand upon the instructions for contributing to this repository.

#TLDR
I have drafted some potential changes to include based on the consistent advice that I have read within the comments of various issues and pull requests that I have reviewed. That said, I am hoping to get feedback from major contributors to ensure that the instructions are indeed following your preferred best practices for contributing.

Details

The CONTRIBUTING.md document needs to be updated to reflect some of the directions included in the ROS2 documentation webpage. As a new (potential) contributor I found myself confused when looking to contribute to the rclpy repository, and the Issues and Pull Requests pages directed me to the CONTRIBUTING.md document. However, the only contents of the document are the requirements for the Apache 2 license statements for the files. It would be beneficial to new contributors to outline the process for contributing to this repository directly within the files rather than only within the ROS2 documentation pages. This would reduce the time that experienced contributors spend re-explaining the same directions to first-time contributors.

Sections to Include

  • Current File Contents: Current License Information and Citation Statement
  • Section for Reporting an Issue
    - Issue Postings
    - Pull Requests
  • Section for First-Time Contributors
  • ?? What else? Please let me know in the comments below so I can add it to a draft and submit a pull request.
@clalancette
Copy link
Contributor

While I agree with you that the rules for first time contributions aren't well laid out, I think we should approach this differently.

In particular, the same rules apply to (most of) the 150 repositories that are in the ROS 2 core. Thus, I think the correct thing to do here is to update the documentation at http://docs.ros.org/en/rolling/The-ROS2-Project/Contributing.html , and then just add links from this repository (and others) to that documentation.

Would you be willing to open a PR over at https://github.com/ros2/ros2_documentation to start that process?

@KristyWaters
Copy link
Author

Absolutely! Thank you for the feedback.

Is there anything else that you think should be added to the document before I open the PR?
(I will start with a DRAFT PR per the guidelines for now.)

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

2 participants