-
Notifications
You must be signed in to change notification settings - Fork 1
Post launch refactor #13
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
Conversation
|
Hey @DapperDino would you be able to break down at a high level what was changed, or which files we can pay attention to/ignore? I think if you request a review from Copilot it also generates a summary based on the changes (though not sure how it'll handle large diffs like this one) |
Mainly updaing code styling, fixing typos, incorrect asset paths, etc... In a couple cases I swapped to using the Nakama package directly, rather than relying on the Asset Store, but now that Nakama has been approved on the Asset Store, I will swap those back now. |
novabyte
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DapperDino Looks good to me 👍
No description provided.