Skip to content

Latest commit

 

History

History
86 lines (54 loc) · 5.68 KB

CONTRIBUTING.md

File metadata and controls

86 lines (54 loc) · 5.68 KB

Contributing to RocketMap

Table Of Contents

I just have a question/need help

How do I get started?

Reporting code issues

Contributing Code

I just have a question/need help.

Please do not open a GitHub issue for support or questions. They are best answered via Discord. GitHub issues are for RocketMap code issues only.

How do I get started?

RocketMap has two main dependencies to get started:

  1. RocketMap is a Python2 project, therefore, you need Python 2!
  2. RocketMap also requires NodeJS in order to build frontend assets.

Python Packages

All the required packages to operate RocketMap are found in requirements.txt in the project root, and can easily be installed by executing pip install --upgrade -r requirements.txt.

NodeJS Packages

NodeJS packages are found in package.json in the project root, and can be easily installed by simply executing npm install.

Reporting code issues

Before you report

  1. Make sure you are on the latest version! When in doubt, git pull.
  2. Ensure it is an actual issue with the project itself. Confirm that you do not have a faulty config, and that you have properly installed all the required dependencies. READ THE ERROR OUTPUT YOU ARE GIVEN! It may have the solution readily printed for you!
  3. Reproduce. Confirm the circumstances required to reproduce the bug. If specific configuration is required, include the required configuration in your report. Find a friend, spin up a virtual machine or use a different computer and test the issue there. If you do not have the issue on the new machine, it may be indicative that something is wrong with your configuration.
  4. USE GITHUB SEARCH. Search open issues and look for any open issues that already address the problem. Duplicate issues will be closed.

Submitting a helpful bug report

When you begin the process of opening an issue, you will be given a template in the text box that should help guide you through the process. Please fill in as much detail as you can.

  • Expected Behavior - What did you expect to happen? Should the map have handled an exception, or have done something different?
  • Current Behavior - What actually happened? Did the map crash? Did a function behave in a way it was not intended? Include any logs or stack traces here.
  • Possible Solution - If you know how this issue is caused and have a possible solution, please include it.
  • Steps to reproduce - Here is where your “Reproduce” stage comes in. Provide the specific configuration that you used. Frontend issue? Provide instructions on what to click or do on the frontend to cause the issue.
  • Context - Why is this an issue?
  • Your environment - Operating system, python --version, pip --version, etc.

If you are asked for more details, please provide details. Any issues not following the above layout or that is not detailed enough to determine the issue may be subject to closure.

Contributing code

Basic Knowledge

To contribute code to RocketMap, it is heavily advised that you are knowledgeable in Git. At a minimum, you should know how to commit, push, pull and do basic rebasing.

Styleguides

RocketMap follows and adheres to the pep8 standards for Python code, and has established rules for JavaScript in the .eslintrc.json file in the project root. Automatic checks of these standards are performed when pull requests are opened, however you can save some time by running these checks on your local machine.

To check if your Python code conforms to PEP8, you can use the flake8 package (pip install flake8). After making changes, open a terminal in the project root and run flake8 --statistics --show-source --disable-noqa.

To check if your Python code conforms to the eslint rules, you can run npm run lint.

Submitting a helpful pull request

  • Description - Describe in detail the changes you made. If you add or remove specific libraries, frameworks, etc, please list the specific frameworks. Any movements between files, e.g. moving code from runserver.py to utils.py, should be noted as such.
  • Motivation and Context - What’s the need for this change? What issue does it solve? If there’s an open issue, please write this exact phrase: “Fixes #prnumber” This will automatically close the issue when the pull request is merged.
  • How has this been tested? - Please include the details of your test. Scan area, configuration, operating system, python version, any other relevant details.
  • Screenshots - If you have made frontend changes, screenshots are highly advised to give context.
  • Types of changes & Checklist - Please put a x in boxes that apply. Make sure it looks like [x], instead of [x ] or [ x], so you get a nice checkbox.

Collaborating with other contributors

When you have an open pull request and wish to collaborate with other contributors, you may request access to the #pr channel in the RocketMap Discord. To request access, you may send a message to Seb or Thunderfox. Please provide a link to your open pull request with your message.