Skip to content

Fix indentation of static path restriction text #1961

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 1 commit into from
Aug 16, 2025

Conversation

traviscross
Copy link
Contributor

In the const eval chapter, we have a list of kinds of expressions that are allowed. Paths to statics are one kind that is allowed, but have certain restrictions we describe in bullet points. We then have text that clarifies when these bullet points are checked.

This text was outdented beyond the relevant item, which broke up the outer bullet point list and created (momentary) ambiguity about the scope of the rules the language might be affecting.

cc @ehuss

@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Aug 15, 2025
In the const eval chapter, we have a list of kinds of expressions that
are allowed.  Paths to statics are one kind that is allowed, but have
certain restrictions we describe in bullet points.  We then have text
that clarifies when these bullet points are checked.

This text was outdented beyond the relevant item, which broke up the
outer bullet point list and created (momentary) ambiguity about the
scope of the rules the language might be affecting.  Let's fix that.
@traviscross traviscross force-pushed the TC/fix-indentation-of-static-path-restriction branch from 1e45abf to 378fa89 Compare August 15, 2025 05:08
Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Thanks!

@ehuss ehuss added this pull request to the merge queue Aug 16, 2025
Merged via the queue into master with commit 6dad80d Aug 16, 2025
5 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Aug 16, 2025
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