To start, thank you for wanting to contribute. The guidelines are pretty straightforward:
- Explain -- What is your PR changing? Why is it changing that section? How does it change it?
- Compatibility -- Please make sure your PR is cross-compatible between TTT and TTT2 before submitting.
- Clean code -- Ensure your code is clean and concise. If you add complex functions, use Lua LS's annotation style to document your functions.
- Optimization -- Most important. If your changes hinder performance significantly, it should be optimized before submission.
Some basic rejection criteria, excluding the obvious above points:
- No explanation -- If you make changes without explaining why (given it's not self-explanatory), your PR will likely be rejected.
- Opinion -- Pull requests may be subject to opinion, so if you're significantly changing how the bots play, please justify your reasons. If it contradicts how I want the bots to play (e.g., too cheaty), please understand that it may be rejected.