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

Update server.gd #1153

Closed
wants to merge 1 commit into from
Closed

Conversation

GabrielDierks
Copy link

No description provided.

@aaronfranke
Copy link
Member

Spam.

@aaronfranke aaronfranke closed this Jan 8, 2025
@GabrielDierks
Copy link
Author

GabrielDierks commented Jan 8, 2025

Actually this example only works with the change. Can you elaborate why it is marked as spam?

My bad, it was a custom class I created. sorry

@Faless
Copy link
Contributor

Faless commented Jan 8, 2025

@GabrielDierks I think this was flagged as it looked like the typical spam PR (adds a seemingly random letter via GH editor), but there is indeed a typo (in the class name definition though) which was introduced in 9a0c857 .

I've opened #1154 with the correct fix (changing the class_name back to the proper name).

@GabrielDierks
Copy link
Author

Lol. I thought I was completely tripping and introduced this typo myself locally. Thanks for clarifying! 🙌

@Calinou Calinou added the bug label Jan 10, 2025
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.

4 participants