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

Revert "Refactor IPC handling in both client and daemon" #339

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

LGFae
Copy link
Owner

@LGFae LGFae commented Jul 1, 2024

Reverts #333.

@rkuklik, I've never done this before, is this what you meant (I just clicked the "revert" button in the PR)?

@rkuklik
Copy link
Contributor

rkuklik commented Jul 1, 2024

To be honest, neither did I. Maybe branch this into separate branch, because I don't think that on the whole the changes are incorrect, just some random thing either in IpcSocket or Daemon, so after my vacation I would fix it and continue with serde rework.

The tests ran successfully and #331 worked, so this should do it. Sorry for the mess, I didn't realize that people can automatically use main without manual install.

@LGFae
Copy link
Owner Author

LGFae commented Jul 1, 2024

Sorry for the mess, I didn't realize that people can automatically use main without manual install.

Eh, I broke the main branch many times before. We just have way more users now. Don't worry about it and enjoy your vacation.

@LGFae LGFae merged commit 2da1f57 into main Jul 1, 2024
10 checks passed
@LGFae LGFae deleted the revert-333-main branch July 1, 2024 18:42
@LGFae
Copy link
Owner Author

LGFae commented Jul 1, 2024

We can simply pull the diff from #331 later and fix whatever was wrong with it.

@rkuklik
Copy link
Contributor

rkuklik commented Jul 1, 2024

Sure. Thank you.

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.

None yet

2 participants