From 3192732fa626bc5708e2dd1582a22977a475b75c Mon Sep 17 00:00:00 2001 From: Allink Date: Thu, 19 May 2022 19:47:53 +0100 Subject: [PATCH] Revert "Revert "Revert "Revert "Revert "Revert "We use Google Code Style in this module"""""" This reverts commit 6db2b56a6933a23fb91f9d6a8bf358019f39471f. --- 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.