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

Deleting unnecessary include in GDScriptParser #83050

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Deleting unnecessary include in GDScriptParser #83050

merged 1 commit into from
Oct 10, 2023

Conversation

k0T0z
Copy link
Contributor

@k0T0z k0T0z commented Oct 9, 2023 •
edited
Loading

While working on #83008, I found an unnecessary include here.

@k0T0z
removing unnecessary include of gdscript_warning.h as it is already i…
3e57279 
…ncluded in the header file

Signed-off-by: Saif Kandil <74428638+k0T0z@users.noreply.github.com>
@k0T0z k0T0z requested a review from a team as a code owner October 9, 2023 15:02
@akien-mga akien-mga changed the title Deleting unnecessary include Deleting unnecessary include in GDScriptParser Oct 9, 2023
@akien-mga akien-mga added this to the 4.2 milestone Oct 9, 2023
@akien-mga akien-mga merged commit 53c6c38 into godotengine:master Oct 10, 2023
15 checks passed
@akien-mga
Copy link
Member

akien-mga commented Oct 10, 2023 •
edited
Loading

Thanks! And congrats for your first merged Godot contribution 🎉

@k0T0z
Copy link
Contributor Author

k0T0z commented Oct 10, 2023

@akien-mga Thank you 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akien-mga akien-mga akien-mga approved these changes

Assignees
No one assigned
Labels
enhancement topic:buildsystem topic:gdscript
Projects
None yet
Milestone
4.2
Development

Successfully merging this pull request may close these issues.
2 participants
@k0T0z @akien-mga