1
0
mirror of https://github.com/laurent22/joplin.git synced 2024-12-21 09:38:01 +02:00
joplin/readme/gsoc2021/pull_request_guidelines.md

22 lines
2.7 KiB
Markdown
Raw Normal View History

2021-03-11 15:54:10 +02:00
# Pull request guidelines
2021-02-07 21:04:49 +02:00
Due to our limited resources and in order to give everyone a chance to submit a pull request, we have put restrictions in place this year. If you want to submit a pull request, please take into account the following rules:
1. Pull requests must be based on an issue that existed **before GSoC was started**, or based on an issue created by a moderator.
2. Each contributor **may only create one pull request at a time**. In some rare cases, once your pull request has been merged, you may be allowed to post a second one, in which case we will let you know. We have this rule in place due to our limited resources - if everyone was allowed to post multiple pull requests we will not be able to review them properly. It is also better for you because you only need to care about one PR - so spend time making sure it is as good as it can be - make sure it works well, has test units, documentation and screenshots (if relevant).
3. If the pull request has serious issues, or would require a significant rewrite to be acceptable, we might closed it and you will not be allowed to open a new one. So **please be careful when posting a PR**.
4. **If you are borrowing code, please disclose it**. It is fine and sometimes even recommended to borrow code, but we need to know about it to assess your work. If we find out that your pull request contains a lot of code copied from elsewhere, we will close the pull request.
5. **All pull request must have test units**. If for some reason it is not possible to add tests, please let us know and explain why.
6. **No Work In Progress**. ONLY completed and working pull requests, and with test units, will be accepted. A WIP would fall under rule 3 and be closed immediately.
2021-03-10 13:13:00 +02:00
7. Please **do not `@mention` contributors and mentors**. Sometimes it takes time before we can review your pull request or answer your questions but we'll get to it sooner or later. `@mentioning` someone just adds to the pile of notifications we get and it won't make us look at your issue faster.
2021-03-11 15:54:10 +02:00
8. **Do not force push**. If you make changes to your pull request, please simply add a new commit as that makes it easy for us to review your new changes. If you force push, we'll have to review everything from the beginning. Additionally, force push often leads to situation where unrelated changes from other commits get added to the pull request, and you need to avoid this. As with the other rules, if we need to ask you multiple times to fix your pull request, we might have to close it as we cannot dedicate that much time dealing with basic git operations.
2021-02-07 21:04:49 +02:00
These rules we hope are fair to everyone, to contributors and maintainers, however if something is unclear or you have any question about them, please let us know!