Skip to content

Latest commit

 

History

History
83 lines (57 loc) · 3.58 KB

CONTRIBUTING.md

File metadata and controls

83 lines (57 loc) · 3.58 KB
Raw

Contributing to RoactRodux

Thanks for considering contributing to RoactRodux! This guide has a few tips and guidelines to make contributing to the project as easy as possible.

Bug Reports

Any bugs (or things that look like bugs) can be reported on the GitHub issue tracker.

Make sure you check to see if someone has already reported your bug first! Don't fret about it; if we notice a duplicate we'll send you a link to the right issue!

Feature Requests

If there are any features you think are missing from RoactRodux, you can post a request in the GitHub issue tracker.

Just like bug reports, take a peak at the issue tracker for duplicates before opening a new feature request.

Working on RoactRodux

To get started working on RoactRodux, you'll need:

Make sure to clone the repository with submodules. You can do that to your existing repository with:

git submodule update --init

Finally, you can run all of RoactRodux's tests with:

lua test/lemur.lua

Or, to generate a LuaCov coverage report:

lua -lluacov test/lemur.lua
luacov

If you're an engineer at Roblox, you can skip this setup and use Roblox-CLI. Make sure it's on your PATH and that you have a production Roblox Studio installation. You can then run:

./test/roblox-cli.sh

Pull Requests

Before starting a pull request, open an issue about the feature or bug. This helps us prevent duplicated and wasted effort. These issues are a great place to ask for help if you run into problems!

Before you submit a new pull request, check:

  • Code Style: Match the existing code!
  • Changelog: Add an entry to CHANGELOG.md
  • Code Quality: Run Selene on your code, no warnings allowed!
  • Code Style: Run StyLua on your code so it's formatted to follow the Roblox Lua Style Guide
  • Tests: They all need to pass!
  • Coverage: Code coverage of tests should not decrease.

Code Style

Try to match the existing code style! In short:

  • Tabs for indentation
  • Double quotes
  • One statement per line

Use StyLua to automatically format your code to comply with the Roblox Lua Style Guide. You can run this tool manually from the commandline, or use one of StyLua's editor integrations.

Changelog

Adding an entry to CHANGELOG.md alongside your commit makes it easier for everyone to keep track of what's been changed.

Add a line under the "Current master" heading. When we make a new release, all of those bullet points will be attached to a new version and the "Current master" section will become empty again.

Tests

When submitting a bug fix, create a test that verifies the broken behavior and that the bug fix works. This helps us avoid regressions!

When submitting a new feature, add tests for all new functionality. Net code coverage of tests should not decrease.

We use LuaCov for keeping track of code coverage. We'd like it to be as close to 100% as possible, but it's not always easy.