enUS Localization Update Announcement / localization-update-announcement (push) Canceled after 0s
1.7 KiB
1.7 KiB
Contributing to Bagshui
Here’s how you can get involved and help make Bagshui better!
Bug reports and feature requests
- Report bugs and request features for this fork on this repository's Issues tab. Please include your ClassicAPI version (
/script DEFAULT_CHAT_FRAME:AddMessage(tostring(CLASSIC_API_VERSION))). - Problems that also happen on Vanilla / WoW 1.18.1 Clients without ClassicAPI may belong upstream instead.
Code changes
All changes happen through pull requests
Please follow Github Flow. TL;DR:
- Fork the repo and create a new branch from
main.- If you have collaborator access, you don’t need to fork, but you do need to create a new branch.
- Make changes.
- Create a pull request.
Please try to follow existing coding style
- Indent using tabs, not spaces (I know, I know).
- Comments that explain the why of the code, not just the what.
- No semicolons at end of lines.
- Avoid collapsing
ifstatements into one-liners.
Don’t do this:if true then return end. - Split long lines (including conditions and function calls).
Try not to go too far past column 120.
License
You agree that your contributions will be under the MIT License.