-
Notifications
You must be signed in to change notification settings - Fork 122
Cross-platform support, completely refactor the project. #18
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
base: main
Are you sure you want to change the base?
Conversation
Co-Authored-By: VashBaldeus <vashbaldeus@gmail.com>
Co-Authored-By: stevebroskey <946790+stevebroskey@users.noreply.github.com>
anyway, reading and writing are done.
|
I don't own a PS5, so I can't test the UART communication code. If anybody is willing to help me out with that, please let me know. |
|
Don’t have an open ps5 to test on but lmk if you would like any help |
|
Hey @PurelyAndy, I've taken a brief look at the refactored code and agree that this is the direction we should be heading in. This fork is very well organized and solves many problems the current main branch has. I'm attempting to merge a small change I made but I don't have write access to your fork. Would you be able to provide that for me? Thanks :3 |
|
This is good. We still need to clean up https://github.com/PurelyAndy/PS5NorModifier/blob/main/UART-CL_By_TheCod3r/Program.cs 's monolithic class |
|
Yeah, I'm working on that right now. There's a lot of shared code between the GUI and CLI apps that needs to be consolidated. |
|
@benjamin238 I'll give you access now. It's in your email. |
|
Why downgrade from .NET 9 to .NET 8 out of curiosity? Could it not be compiled as Native AOT? Avalonia supports it: https://docs.avaloniaui.net/docs/deployment/native-aot |
|
.NET 8 is LTS. I don't know how often they want to update the .NET version of this project. It's trivial to update it back to .NET 9 or 10 in the future, anyway. |
Internally document model codes, rename some variables with unclear names, resolved dependency problems (wouldn't compile without this)
|
I've also submitted a PR for a rewrite and to clean up the CLI, but I took a more drastic approach. @PurelyAndy if you want to take a look #33 |
I'm rewriting the GUI with Avalonia and restructuring the projects, moving shared code, cleaning up, etc.