Skip to content

merge#17

Open
axiev1 wants to merge 50 commits into
hellofrom
main
Open

merge#17
axiev1 wants to merge 50 commits into
hellofrom
main

Conversation

@axiev1

@axiev1 axiev1 commented Jan 19, 2025

Copy link
Copy Markdown
Collaborator

Please fill out the following before requesting review on this PR

Description

Testing Done

Resolved Issues

Length Justification and Key Files to Review

Review Checklist

It is the reviewers responsibility to also make sure every item here has been covered

  • Function & Class comments: All function definitions and classes should have some sort of basic JavaDoc comments. use /// to generate XMLDocs for C# using IDE.
  • Remove all commented out code
  • Remove extra print statements: for example, those just used for testing
  • Resolve all TODO's: All TODO (or similar) statements should either be completed or associated with a github issue
  • Play Mode Runs

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