From d533bf195ffd0270e508b6dcdcb5126587aa4117 Mon Sep 17 00:00:00 2001 From: Allink Date: Thu, 19 May 2022 19:52:17 +0100 Subject: [PATCH] Revert "Revert "Revert "Revert "Revert "Revert "Revert "Revert "Revert "Revert "We use Google Code Style in this module"""""""""" This reverts commit 2bf5346be23d1738a7eb9bc7796226843fdd9356. --- CONTRIBUTING.md | 24 ++++++------------------ 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index fd0f99d..406129d 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,36 +1,24 @@ -For those who are wanting to contribute, we fully encourage doing so. There are a few rules we require following when -contributing however. +For those who are wanting to contribute, we fully encourage doing so. There are a few rules we require following when contributing however. ## Steps - 1. Make an issue and get feedback. It's important to know if your idea will be accepted before writing any code. - - If it is a feature request, describe the feature and be extremely specific. - If it is a bug report, ensure you include how to reproduce the bug and the expected outcome - If it is an enhancement, describe your proposed changes. Ensure you are extremely specific. - 2. Fork this project -3. Create a new branch that describes the new feature, enhancement, or bug fix. For example, this is - good: `feature/add-xyz`. This is bad: `fix-this-lol`. +3. Create a new branch that describes the new feature, enhancement, or bug fix. For example, this is good: `feature/add-xyz`. This is bad: `fix-this-lol`. 4. Write the code that addresses your change. - -- Keep in mind that it **must** be formatted correctly. If you are using IntelliJ, there is a `codeStyle.xml` file that - tells IntelliJ how to format your code. Check this link for information on how to use the - file: https://www.jetbrains.com/help/idea/configuring-code-style.html#import-export-schemes -- If you are not using IntelliJ, that is fine. We use Allman style so please format your code accordingly. - +- Keep in mind that it **must** be formatted correctly. If you are using IntelliJ, there is a `codeStyle.xml` file that tells IntelliJ how to format your code. Check this link for information on how to use the file: https://www.jetbrains.com/help/idea/configuring-code-style.html#import-export-schemes +- If you are not using IntelliJ, that is fine. We use Google Java Code Style in this particular Module so please format your code accordingly. 6. Push your changes to your new branch and make a PR based off of that branch. ## Requirements for a PR - - The issue must be marked as approved - It must only address each specific issue. Don't make one PR for multiple issues. - Your PR must compile and work. If it does not compile or work, your PR will most likely be rejected. ## Code requirements - -- Most importantly, your code must be efficient. Your pull request may be rejected if your code is deemed inefficient or - sloppy. +- Most importantly, your code must be efficient. Your pull request may be rejected if your code is deemed inefficient or sloppy. - Do not repeat yourself. Create functions as needed if you're using large blocks of code over and over again. - Do not use an excessive amount of commits when making your PR. It makes the master branch look messy. -- Your code must be consistent with Plex's codebase. If a function already exists, use it. \ No newline at end of file +- Your code must be consistent with Plex's codebase. If a function already exists, use it.