From 01a328563675c3166540716a017abfc74c99f792 Mon Sep 17 00:00:00 2001 From: Laurent Cozic Date: Fri, 25 Jan 2019 17:14:17 +0000 Subject: [PATCH] Updated doc --- CONTRIBUTING.md | 10 ++++++---- docs/PULL_REQUEST_TEMPLATE | 2 -- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 4b4a72acb..8daf30042 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -16,14 +16,16 @@ If possible, **please provide a screenshot**. A screenshot showing the problem i Again, please check that it has not already been requested. If it has, simply **up-vote the issue** - the ones with the most up-votes are likely to be implemented. "+1" comments are not tracked. -# Adding new features +# Creating a pull request -If you want to add a new feature, consider asking about it before implementing it or checking existing discussions to make sure it is within the scope of the project. Of course you are free to create the pull request directly but it is not guaranteed it is going to be accepted. +- If you want to add a new feature, consider asking about it before implementing it or checking existing discussions to make sure it is within the scope of the project. That scope, due to limited resources, might be narrower than you think. As a rule of thumb **if your change is likely to involve more than 50 lines of code, you should discuss it in the forum**, just so that you don't waste your time implementing something that might not be accepted. + +- Bug fixes have a very high change of being accepted. + +- A pull request that is relevant to the current roadmap has a very high change of being accepted. Building the apps is relatively easy - please [see the build instructions](https://github.com/laurent22/joplin/blob/master/BUILD.md) for more details. -Pull requests that automatically change many files tend to be rejected (for example changes that automatically update the code styling, or to add new lines to many files, or to automatically convert images to a different format) so if you have such a pull request in mind, please discuss it first in the forum. - # Coding style There are only two rules, but not following them means the pull request will not be accepted (it can be accepted once the issues are fixed): diff --git a/docs/PULL_REQUEST_TEMPLATE b/docs/PULL_REQUEST_TEMPLATE index 04cf1b38f..e9b04468d 100644 --- a/docs/PULL_REQUEST_TEMPLATE +++ b/docs/PULL_REQUEST_TEMPLATE @@ -1,7 +1,5 @@