From 7478cad1b3130ad0f206f1ac30de6a98b531ff76 Mon Sep 17 00:00:00 2001 From: Francesco Borzì Date: Sun, 3 Mar 2019 18:04:16 +0100 Subject: Update and rename to best-practices.md --- docs/best-practices.md | 35 +++++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) create mode 100644 docs/best-practices.md (limited to 'docs/best-practices.md') diff --git a/docs/best-practices.md b/docs/best-practices.md new file mode 100644 index 0000000..a9f19af --- /dev/null +++ b/docs/best-practices.md @@ -0,0 +1,35 @@ +# Best Practices + +A collection of best practices to use AzerothCore. + + +## Custom changes + +- **NEVER** add custom changes to the AzerothCore sources. [Create modules](http://www.azerothcore.org/wiki/Create-a-Module) instead. + +If you need new hooks to implement your custom changes into modules, feel free to send a PR that implements them. + +Reasons: + +1. keeping your base source code clean will make it easier for you to update it +2. keeping your customizations into modules will allow you to easily enable/disable them to troubleshoot any potential issue +3. modularized software is much easier to maintain + + +## Pull Requests (PR) + +- [Sync your fork](http://www.azerothcore.org/wiki/Syncing-your-fork) before opening a PR + +Reasons: + +1. Helps preventing merge conflicts +2. You will implement (and test) your changes as they were applied to latest master version + +- **NEVER** push changes to the `master` branch. Aalways your `master` branch clean. + +When you create a new PR, do `git checkout master` and create a new branch using `git checkout -b new-branch`. + +Reasons: + +1. This will allow you to create multiple PRs that are independent from each other +2. This will allow you to easily update your branches with latest master by first synching your fork and then merging `master` into your PR branches. -- cgit