-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Docs: List overview separate from tutorials #2878
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
Conversation
for more information, see https://pre-commit.ci
Personally, I think this is clearer. Does everyone agree? |
|
Hey Dhiraj, awesome job on creating the PR! This is perfect. I see the maintainer (Ewout) has already reviewed it and left a positive comment. The automatic checks have all passed, which is great. For now, your work is done! You just need to wait for one or two maintainers to formally approve it, and then they will merge your changes. Excellent first contribution! |
|
thank u so much |
|
I thin is is a nice improvement |
updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.2 → v0.14.3](astral-sh/ruff-pre-commit@v0.14.2...v0.14.3) - [github.com/asottile/pyupgrade: v3.20.0 → v3.21.0](asottile/pyupgrade@v3.20.0...v3.21.0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
EwoutH
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, it now looks good!
|
Congratulations on getting your first PR merged! |
This PR updates the Getting Started page to list the Overview section separately from the Tutorials. It clarifies that: - The Overview gives a passive introduction to Mesa’s features and structure. - The Tutorials are for hands-on, step-by-step learning.

This PR updates the Getting Started page to list the Overview section separately from the Tutorials.
It clarifies that:
Fixes #2876