Skip to content

Fix grammar links #1851

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

Merged
merged 4 commits into from
Jun 17, 2025
Merged

Fix grammar links #1851

merged 4 commits into from
Jun 17, 2025

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Jun 11, 2025

This fixes some links to grammar production names. Some of these links were using the wrong name. Also, the explicit link definitions aren't necessary anymore.

ehuss added 3 commits June 11, 2025 11:14
These links were using the wrong name (_LazyBooleanOperatorExpression_)
which doesn't exist.
This was missing an `s` as a plural.
Now that mdbook-spec auto-generates link definitions for grammar names,
there's no need to be explicit about these.
@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Jun 11, 2025
In rust-lang#1787 I missed linkify-ing
references to grammar rules that weren't links. This makes sure that
they are linked and validated.
@traviscross traviscross added this pull request to the merge queue Jun 17, 2025
Merged via the queue into rust-lang:master with commit 5238a8f Jun 17, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: The marked PR is awaiting review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants