Skip to content

Documentation update#6298

Open
montplaisir wants to merge 2 commits into
developfrom
fix-update-documentation
Open

Documentation update#6298
montplaisir wants to merge 2 commits into
developfrom
fix-update-documentation

Conversation

@montplaisir

Copy link
Copy Markdown

Proposed change(s)

This PR addresses open PRs about ML-Agents documentation.

The docs directory is not used anymore. Updates were done to files in com.unity.ml-agents/Documentation~. The files in docs were removed, except for redirecting to the updated documentation.

The following PRs are integrated:
#5705 fix typo
#6184 Missing reference in code example (doublon of #5705)
#5566 Add tips in docs
#6110 Update Installation.md

The following PRs were already fixed:
#6205 Updated Readme.md to have the correct link to the CODE_OF_CONDUCT.md
#6148 fix: invalid image path

This PR also fixes some typos, including one found in #5646.

Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)

#5566
#5646
#5705
#6110
#6148
#6184
#6205

Types of change(s)

  • Bug fix
  • New feature
  • Code refactor
  • Breaking change
  • Documentation update
  • Other (please describe)

Checklist

  • Added tests that prove my fix is effective or that my feature works
  • Updated the changelog (if applicable)
  • Updated the documentation (if applicable)
  • Updated the migration guide (if applicable)

Other comments

The `docs` directory is not used anymore. Updates were done to files in `com.unity.ml-agents/Documentation~`. The files in `docs` were removed, except for redirecting to the updated documentation.

The following PRs are integrated:

The following PRs were already fixed:

This PR also fixes some typos, including one found in #5646.
@CLAassistant

CLAassistant commented Jun 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Codengineer Codengineer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice cleanup!

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

Successfully merging this pull request may close these issues.

3 participants