Skip to content

[doc] trivial: Fix broken developer links #54942

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Jul 26, 2025
Merged

[doc] trivial: Fix broken developer links #54942

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion doc/source/ray-contribute/getting-involved.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ What can I work on?
-------------------

We use Github to track issues, feature requests, and bugs. Take a look at the
ones labeled `"good first issue" <https://github.com/ray-project/ray/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22>`__ for a place to start.
ones labeled `"good first issue" <https://github.com/ray-project/ray/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+label%3A%22good-first-issue%22>`__ for a place to start.

Setting up your development environment
---------------------------------------
Expand Down
2 changes: 1 addition & 1 deletion doc/source/tune/faq.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -491,7 +491,7 @@ How can I get started contributing to Tune?
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

We use Github to track issues, feature requests, and bugs. Take a look at the
ones labeled `"good first issue" <https://github.com/ray-project/ray/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22>`__ and `"help wanted" <https://github.com/ray-project/ray/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22>`__ for a place to start.
ones labeled `"good first issue" <https://github.com/ray-project/ray/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+label%3A%22good-first-issue%22>`__ and `"help wanted" <https://github.com/ray-project/ray/issues?q=is%3Aopen+is%3Aissue+label%3A%22help-wanted%22>`__ for a place to start.
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot Jul 26, 2025

Choose a reason for hiding this comment

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

medium

Thanks for fixing these broken links! Your change is correct as GitHub labels now use hyphens instead of spaces.

While you're at it, we can simplify these URLs further for better readability and maintainability.

  • The utf8=... parameter is a relic from older web forms and is not necessary for the query to work.
  • The is:issue and is:open filters are redundant as they are the default for the repository's issues page.
  • The quotes around the label names are also not required since the label names themselves don't contain characters that need quoting.

Here is a cleaner version of the links:

ones labeled `"good first issue" <https://github.com/ray-project/ray/issues?q=label:good-first-issue>`__ and `"help wanted" <https://github.com/ray-project/ray/issues?q=label:help-wanted>`__ for a place to start.

Copy link
Contributor Author

@isimluk isimluk Jul 26, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Thank You for all the suggestion that you so kindly put together.

However, please appreciate that my ambition is to fix human visible issue -- and not to optimize length of the URL.

Also, I think your suggestion about 'is:open' filter being default is not correct as of today.

Look for issues with "[tune]" in the title.

.. note::
Expand Down