[docs] développeurs: vérifications avant la demande de fusion d'une merge request
Compare changes
+ 46
− 2
@@ -42,6 +58,8 @@ For example, we rename the trait `Foo` to `Fii` in the `durs-crate` crate:
@@ -42,6 +58,8 @@ For example, we rename the trait `Foo` to `Fii` in the `durs-crate` crate:
@@ -56,6 +74,8 @@ For example, we rename the trait `Foo` to `Fii` in the `durs-crate` crate:
@@ -56,6 +74,8 @@ For example, we rename the trait `Foo` to `Fii` in the `durs-crate` crate:
@@ -77,6 +97,7 @@ Every time the `dev` branch is updated, you must rebase each of your working bra
@@ -77,6 +97,7 @@ Every time the `dev` branch is updated, you must rebase each of your working bra
4. When you don't have any conflict anymore after `git rebase --continue`, then the rebase succeeded. Then rebase a remaning branch.
@@ -88,12 +109,35 @@ You must prefix your commit with `wip:` when it is a work in progress.
@@ -88,12 +109,35 @@ You must prefix your commit with `wip:` when it is a work in progress.
In case of conflict during rebase that you can not solve, contact a lead developer telling him the hash of the commit on which YOUR_BRANCH is currently based so he can reproduce the rebase and see the conflicts. While waiting for his answer, you can cancel the rebase and work on YOUR_BRANCH without updating:
If there are no conflicts or if they have been resolved, you can request the merging of your MR (if not, do not do so). Once the merge request has been issued, a discussion is opened specifically for it. It will allow you to discuss the changes you have made. Feel free to tag someone by writing @pseudo so that they are notified of your request. Don't be impatient, the review of your contribution may take more or less time depending on its content! It is better to take your time before integrating a new contribution because once integrated into the dev branch, changing commits would be very annoying for other contributors. This helps to keep a clear and understandable commit history.
The general discussion is used to comment on the merge request as a whole, for example to tag a developer for a proofreading request. When it comes to discussing a specific change in the code, you should go to the "Changes" tab of the merge request and comment under the code extract involved. This makes it easier to break down the resolution of problems raised by the merge request via the "comment resolution" feature.