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

Syntax highlighting changes #515

Merged

Conversation

ryanabx
Copy link
Contributor

@ryanabx ryanabx commented Oct 16, 2023

NOTE: This is my first time working with regular expressions, so please let me know if I've made a mistake with my added regular expressions!

Before:
image

After:
image

@atirut-w
Copy link

The new annotation should probably be in a separate PR that depends on godotengine/godot#82987 being merged. The other two seems great for consistency.

@ryanabx ryanabx force-pushed the syntax/highlighting-changes branch 2 times, most recently from 956dc99 to 23557f0 Compare October 16, 2023 13:10
@DaelonSuzuka
Copy link
Collaborator

DaelonSuzuka commented Oct 16, 2023

The new annotation should probably be in a separate PR that depends on godotengine/godot#82987 being merged.

No, there's no reason to push that out to another PR. Adding and removing annotations is a trivial change, and the linked engine PR seems quite likely to get merged.

The grammar in this extension is explicitly attempting to be a superset of all versions of GDScript, so adding support for future features is just as valid as maintaining support for Godot 3 features or features that were deprecated and removed.

@ryanabx ryanabx force-pushed the syntax/highlighting-changes branch from 23557f0 to 00a7f9c Compare October 19, 2023 19:44
Better syntax highlighting for properties. Class variables used to be unhighlighted and are now highlighted properly as `variable.other.gdscript`, properties are still highlighted as `variable.other.property.gdscript`
@ryanabx ryanabx force-pushed the syntax/highlighting-changes branch from 00a7f9c to 211fbf3 Compare October 19, 2023 19:45
@DaelonSuzuka
Copy link
Collaborator

Sorry for the delay. Everything looks fine to me.

@DaelonSuzuka DaelonSuzuka merged commit dcbfcb4 into godotengine:master Oct 22, 2023
1 check passed
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