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

Adds Multiplayer #16

Merged
merged 12 commits into from
Oct 3, 2023
Merged

Conversation

Superyodeler
Copy link
Contributor

About The Pull Request

Adds multiplayer to the game. There are a few things that could be cleaned up a bit, but these can be done without another major rework, and I think this is in a good enough state to add now so that new changes don't have to be ported over.

Why It's Good For The Game

The sooner we add multiplayer, the fewer major features we have to painstakingly port over.

Changelog

  • Added Multiplayer
  • Added Complexity

Superyodeler and others added 8 commits September 2, 2023 22:58
…and a shared folder handling the simulation logic and will also be used by the server.

This commit doesn't include multiplayer, it's just the initial split of the code.


Co-authored-by: Mark Suckerberg <[email protected]>
Co-authored-by: Imaginos16 <[email protected]>
Co-authored-by: Zeta <[email protected]>
…me.git into Multiplayer

# Conflicts:
#	Blocktest/Code/Scenes/GameScene.cs
…me.git into Multiplayer

# Conflicts:
#	Blocktest/Code/Scenes/GameScene.cs
…r the server. Also removes a couple of logging statements.
@Superyodeler
Copy link
Contributor Author

Note: I will need someone to test this on Windows.

@Superyodeler
Copy link
Contributor Author

I've added this to the readme, and I'll add it here too: To run blocktest without networking should work the same as before, but to connect to a server you need to use command line args atm. The args are "connect " or "connect localhost" for a server running on the same machine.

@MarkSuckerberg MarkSuckerberg merged commit ee3c95e into blocktest-game:master Oct 3, 2023
@Imaginos16
Copy link
Contributor

W

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