You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[s1949] We centralized our issue tracking on cmangos/issues
All existing issues have been migrated to this centralized repository. As
of now, we will deactivate the issue tracker on this repository and only use
the new central one.
Please note that Pull Requests are still on the repository containing the
code and you'll still post them there, as they are directly associated
with that code.
Thanks to @github for making this possible with
https://github.com/blog/1439-closing-issues-across-repositories
[ci skip]
(based on commit [12426] - e7ecc5f)
Signed-off-by: DasBlub <[email protected]>
Copy file name to clipboardexpand all lines: CONTRIBUTING.md
+20-18
Original file line number
Diff line number
Diff line change
@@ -18,7 +18,8 @@ which is hosted at https://github.com/cmangos
18
18
19
19
* Make sure you have a [Forum account](http://cmangos.net)
20
20
* Make sure you have a [GitHub account](https://github.com/signup/free)
21
-
* Fork the CMaNGOS repository on Github (This way you will get informed about changes by github)
21
+
* Watch the CMaNGOS repository on GitHub (This way you will get informed about changes by GitHub)
22
+
* Optionally: Fork the CMaNGOS repository on GitHub in order to be able to publish own changes and create pull requests with them (see below for details)
22
23
23
24
You can help us developing the core by (at least!)
24
25
* Giving Feedback on Bugs
@@ -27,7 +28,7 @@ You can help us developing the core by (at least!)
27
28
28
29
# Bug-Reports
29
30
30
-
If you spot a bug, you can either report it on our forums (cmangos.net) or by opening an issue in one of our github repositories.
31
+
If you spot a bug, please report it by opening an issue on our centralized GitHub issue tracking repository: [cmangos/issues](https://github.com/cmangos/issues/issues).
31
32
Before you do so, please consider the following points:
32
33
33
34
* The bug happens on a clean core
@@ -36,18 +37,18 @@ Before you do so, please consider the following points:
36
37
37
38
When you decide to create a new bug report, please make sure that this report includes:
38
39
39
-
*On which revision the bug happens
40
-
* Revisions of additional parts of the CMaNGOS familiy (SD2-version, Database Name and Version, and such)
41
-
*What you need to do do trigger the bug, be as detailed as possible.
42
-
Please use spell-ids in your description, and try to use .cast commands to reproduce your bug. This makes reproducing much easier!
43
-
*What IS happening WHEN doing your actions
44
-
*What SHOULD happen when doing your actions - if possible also add a source why you think this should happen
45
-
46
-
*Please report only one bug per issue/thread!
40
+
*Revision on which you encountered this bug (if known, also the revision or timeframe where it first appeared) - you can also post the link to the commit on the upstream repository
41
+
* Revisions of additional parts of the CMaNGOS familiy (SD2version, database name and version, and such)
42
+
*A detailed step-by-step report how this bug can be reproduced
43
+
* A detailed explenation of what DOES happen
44
+
*A detailed explenation of what SHOULD happen - if possible also add a source why you think this should happen
45
+
*When reporting spell-related bugs, please use spell-IDs in your description, and try to use .cast commands to reproduce your bug. This makes reproducing much easier!
46
+
* When reporting crashes, please include a crash-log (as detailed as possible - we like GDB logs from debug builds!)
47
+
*Only report one bug per issue/thread
47
48
48
49
# Testing features in development
49
50
## Importing pull-requests
50
-
Merging pull-requests into your branch for testing can be done by the GUIs provided by Github.
51
+
Merging pull-requests into your branch for testing can be done by the GUIs provided by GitHub.
51
52
For people not wanting to use them, we have a "MergeHelper.sh" script shipped in contrib/
52
53
53
54
You can - also on Windows within a Git Bash - start the script by using
@@ -67,7 +68,7 @@ which will simply merge pull-request 37 from your origin repository
67
68
68
69
This should be especially reasonable, if you yourself use topic branches like described in the next section!
69
70
70
-
Please be sure to feedback your test-results through proper channels -- this means commenting on the pull-requests, commenting on issues and commits on github.
71
+
Please be sure to feedback your test-results through proper channels -- this means commenting on the pull-requests, commenting on issues and commits on GitHub.
71
72
72
73
73
74
## Importing features from third-party repos
@@ -109,20 +110,21 @@ This code style is automatically enforced by the helper scripts in contrib/clean
109
110
110
111
You can publish and share your contribution projects via forum or by pull-requests
111
112
112
-
### Via Forum
113
-
Using the our forum for sharing patches is perfectly fine, but for complicated features we strongly suggest to use topic branches and pull-requests
114
-
If you want to use the forum, please consider _also_[Forum Patch submission Guidelines](http://cmangos.net/thread-42.html)
115
-
116
-
### Via Pull-Requests on Github
117
113
When you have created your topic-branch and pushed it to your local repository, you can easily create a pull-request to our repo from this!
114
+
Please always create the pull request on the repository containing the CMaNGOS code you were working on.
118
115
119
-
Here is the [Github Information on Pull-Requests](https://help.github.com/articles/using-pull-requests)
116
+
Here is the [GitHub Information on Pull-Requests](https://help.github.com/articles/using-pull-requests)
120
117
121
118
This has these advantages:
122
119
* Your code will be read and commented on by many people
123
120
* Your code will be fetched by people willing to help testing
124
121
* Once you have opened a pull-request for a branch, every change of the branch (like a new commit) will automatically be added to the pull-request.
125
122
So you won't have any additional overhead in maintaining the pull-request :)
123
+
* Travis CI will automatically test the build of your pull request and report the status just as it does for our commits
124
+
125
+
If it is only a minor change and you do not want to create a topic-branch and a pull request (though you should, it's fast and easy to do!),
126
+
you can also open a normal issue and paste the diff in there (but please use the Markdown tags to mark it as a code!). Rule of thumb is that
127
+
more than 20 lines should most definitely be posted as a pull request instead of an issue with a simple diff.
0 commit comments