Add source control best practices guide

[sirishapadmasekhar]

Description

Adds a new Source Control Best Practices guide under the Contributing section.

The guide covers:

• Avoiding generated workspace artifacts in commits
• Editor-specific files
• Using a global gitignore
• Authentication and credential management references

Also adds the page to the Contributing documentation navigation.

AI Usage

ChatGPT was used to assist with reviewing and restructuring the documentation.

Fixes #6040

[fujitatomoya]

description looks good to me once the workflow is green. i would like to have 2nd approval before merge.

CC: @ahcorde @christophebedard @kscottz

[fujitatomoya]

i will start the workflow, so that you can see the failures by lines.

[fujitatomoya]

@sirishapadmasekhar thanks for the PR, can you take a look at https://docs.ros.org/en/rolling/The-ROS2-Project/Contributing/Contributing-To-ROS-2-Documentation.html? the tests and checks can be done in your local environment, please make sure all tests are passing.

[sirishapadmasekhar]

@fujitatomoya Thanks for the review. I updated the document to use one sentence per line and pushed the changes. I also ran the local documentation checks (doc8 and sphinx-lint), and both pass successfully.

[fujitatomoya]

@sirishapadmasekhar thanks for fixing. there is one thing to confirm since you did not use our pull request template, did you use any AI tools such as codex or claunde? if so, can you disclose that information?

[peci1]

I've suggested a few improvements.

Apart from them, I have one more high-level observation: what about changing the structure like this?

• intro
• avoid committing generated and temporary files
  • sources of generated/temporary/backup files
    • colcon workspace artifacts
    • IDE configs
    • editor temporary/backup files
    • OS files (.DS_store)
  • gitignore placement options:
    • repo gitignore
    • global gitignore
• credential helpers
• developing on shared robots
• additional resources

[sirishapadmasekhar]

Thanks @fujitatomoya and @peci1 for the review and suggestions. I have addressed the requested changes, updated the document structure and sentence wrapping, verified that doc8 passes and make html builds successfully, and added the AI usage disclosure to the PR description.
Please let me know if there are any further changes needed.

[fujitatomoya]

i would like to have 2nd approval on this before merge. please wait for a bit.