Skip to content
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

Jump to bottom

Adjust identifiers in inline-assembly.md to be more consistent with other chapters #1635

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adjust identifiers in inline-assembly.md to be more consistent with other chapters #1635

wants to merge 2 commits into from

Conversation

chorman0773
Copy link
Contributor

This adjusts a number of identifiers in the chapter to follow the same patterns used in other PRs introducing identifier syntax, mainly:

  • Many rules were reduced to one or (rarely) two dash separated words
  • The identifier was removed from the example (per Explain how to name rule identifiers #1609 bullet 3 "Rust code tests and examples do not need identifiers")

@chorman0773 chorman0773 mentioned this pull request Oct 8, 2024
Open
@rustbot

This comment has been minimized.

Sign in to view
@rustbot rustbot added the S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@traviscross traviscross Awaiting requested review from traviscross

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chorman0773 @rustbot