Skip to content

Add scoped, not, and, or keyword highlights - #97

Merged
MrSubidubi merged 2 commits into
zed-extensions:mainfrom
alihakankurt:feature/pattern-and-scoped-keywords
Aug 14, 2026
Merged

Add scoped, not, and, or keyword highlights#97
MrSubidubi merged 2 commits into
zed-extensions:mainfrom
alihakankurt:feature/pattern-and-scoped-keywords

Conversation

@alihakankurt

Copy link
Copy Markdown
Contributor

Summary

Adds syntax highlighting for not, or, and, and scoped keywords in the C# tree-sitter highlights.

  • not / or / and: highlight pattern keywords via negated_pattern, or_pattern, and_pattern
  • scoped: highlight the scoped type keyword via scoped_type

Before / After

image image

Test Plan

Opened a .cs file using a pattern expression and verified not, or, and, and scoped are highlighted.

@cla-bot

cla-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @alihakankurt on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@alihakankurt

Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed label Aug 10, 2026
@cla-bot

cla-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

MrSubidubi
MrSubidubi previously approved these changes Aug 14, 2026

@MrSubidubi MrSubidubi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very nice, thanks and congrats to such a good first contribution! 🎉

@MrSubidubi
MrSubidubi enabled auto-merge (squash) August 14, 2026 10:34
Comment thread languages/csharp/highlights.scm
@MrSubidubi
MrSubidubi merged commit 0cefd83 into zed-extensions:main Aug 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants