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

Updated Multiplayer Pong C# to Godot 4.2 #1045

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tinybitofheaven
Copy link

This is my attempt for updating the Multiplayer Pong C# demo project to Godot 4. This demo was marked as needing an update in [#697]. I tested it locally, it works as expected. I kept the comments mostly as is, please let me know if anything about what I made needs to change!

@tinybitofheaven
Copy link
Author

tinybitofheaven commented Apr 17, 2024

I was wondering what the issue was with this formatting check? It isn't telling me exactly what's wrong and as far as I can tell, the lines of code it's flagging in the .sln file are exactly the same.

image

@aaronfranke
Copy link
Member

@tinybitofheaven That file has an invisible BOM at the start of the file, which should be removed.

@tinybitofheaven
Copy link
Author

@aaronfranke Thanks, I believe it is fixed now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants