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.

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

Allow option to highlight operator as as keyword #487

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mweber15
Copy link

@mweber15 mweber15 commented Nov 6, 2022

While as is functionally an operator, it is often highlighted as if it were a keyword. This change gives the user the option to link rustOperatorKeyword to Keyword while preserving the current default behavior. This allows for as to be highlighted like a keyword separately from other operators.

The documentation lists as as a keyword and consistently highlights it as such.

https://doc.rust-lang.org/std/index.html#keywords
https://doc.rust-lang.org/std/keyword.as.html

While `as` is functionally an operator, it is often highlighted as if it
were a keyword. This change gives the user the option to link
`rustOperatorKeyword` to `Keyword` while preserving the current default
behavior. This allows for `as` to be highlighted like a keyword
separately from other operators.

The documentation lists `as` as a keyword and consistently highlights it
as such.

https://doc.rust-lang.org/std/index.html#keywords
https://doc.rust-lang.org/std/keyword.as.html
@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @chris-morgan (or someone else) soon.

Please see the contribution instructions for more information.

@mweber15
Copy link
Author

mweber15 commented Nov 6, 2022

This is following up on #486.

@Iron-E
Copy link

Iron-E commented Dec 10, 2022

This is implemented in #431

@Iron-E Iron-E mentioned this pull request Dec 10, 2022
1 task
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.

None yet

4 participants