Skip to content

v1.3#139

Draft
IExploitableMan wants to merge 13 commits intomainfrom
dev1.3
Draft

v1.3#139
IExploitableMan wants to merge 13 commits intomainfrom
dev1.3

Conversation

@IExploitableMan
Copy link
Copy Markdown
Contributor

@IExploitableMan IExploitableMan commented Mar 15, 2026

No description provided.

@johnklipi
Copy link
Copy Markdown
Contributor

can this bro stop touching my draft pr pls

@johnklipi
Copy link
Copy Markdown
Contributor

actually not even my pr cuz i didnt ask for creating a pr yet

coderabbitai[bot]

This comment was marked as low quality.

@PolyModdingTeam PolyModdingTeam deleted a comment from coderabbitai Bot Mar 26, 2026
@PolyModdingTeam PolyModdingTeam deleted a comment from coderabbitai Bot Mar 26, 2026
@PolyModdingTeam PolyModdingTeam deleted a comment from coderabbitai Bot May 4, 2026
@PolyModdingTeam PolyModdingTeam deleted a comment from coderabbitai Bot May 4, 2026
Comment thread src/Multiplayer/Client.cs
{
Console.Write("GameState_EndPlayerTurn");
__instance.CurrentPlayerIndex++;
if (__instance.CurrentPlayerIndex >=__instance. PlayerStates.Count)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How this even compiles btw?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk can you stop reviewing something i didnt submit for review thx

Comment thread src/Multiplayer/Client.cs
var currentPlayer = __instance.PlayerStates[__instance.CurrentPlayerIndex];
if (!currentPlayer.IsAlive(__instance))
{
__instance.EndPlayerTurn(newTurn);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Recursion?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copied from vanilla code

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