Contributing to Maintainerr¶
All help is welcome and greatly appreciated! If you would like to contribute to the project, the following instructions should get you started... (The below is specific to a Windows Development environment.)
Development¶
Tools Required¶
- HTML/TypeScript/JavaScript editor
- VSCode is recommended. Upon opening the project, a few extensions will be automatically recommended for install.
- NodeJS (Node 20.x or higher)
- Git
Getting Started¶
- Add the remote
upstream
:
- Create a new branch:
-
It is recommended to give your branch a meaningful name, relevant to the feature or fix you are working on.
- Good examples:
docs-docker-setup
feat-new-system
fix-title-cards
ci-improve-build
- Bad examples:
bug
docs
feature
fix
patch
-
Activate the correct Yarn version. (Note: In order to run
corepack enable
, you will need to be running cmd or PowerShell as an Administrator.)
- Install dependencies
- As of Maintainerr v2.0, the project looks to ensure you have read/write permissions on the
data
directory. Thisdata
directory does not exist when you first clone your fork. Before running the below commands, create a folder inside of your main Maintainerr directory nameddata
, and ensure it has full permissions to theEveryone
user.
- Run the development command
-
If the build fails with PowerShell, try to use cmd instead.
-
Make your code changes/improvements and test that they work as intended.
-
Should you need to update your fork (from any recent ORIGIN changes), you can do so by rebasing from
upstream
:
Contributing Code¶
- If you are taking on an existing bug or feature ticket, please comment on the issue to avoid multiple people working on the same thing.
- If you have a major change or large feature to contribute, reach out via Discussions or our Discord server first to align with us! It'll save us all time, as the review will be quicker, and we will have more context about your change, allowing us to guide you in the right direction.
- You can create a "draft" pull request early to get feedback on your work.
- Large PRs (>500 lines changed, excluding tests) should be split into smaller, manageable parts by stacking your PRs sequentially. This approach makes the review process significantly easier when done in order.
- Each PR should target one major meaningful change, which allows us to review independent changes separately, rather than having everything blocked on a single review.
- All commits must follow Conventional Commits
- Pull requests with commits not following this standard will not be merged.
- Please make meaningful commits, or squash them prior to opening a pull request.
- Do not squash commits once people have begun reviewing your changes.
- Always rebase your commit to the latest
main
branch. Do not mergemain
into your branch. - It is your responsibility to keep your branch up-to-date. Your work will not be merged unless it is rebased off the latest
main
branch. - Your code must be formatted correctly.
- We use Prettier to format our code base. It is recommended to have the Prettier extension installed in your editor and to format on save. Alternatively, you can run
yarn format
to format.
UI Text Style¶
When adding new UI text, please try to adhere to the following guidelines:
- Be concise and clear, and use as few words as possible to make your point.
- Use the Oxford comma where appropriate.
- Use the appropriate Unicode characters for ellipses, arrows, and other special characters/symbols.
- Capitalize proper nouns, such as Plex, Radarr, Sonarr, Telegram, Slack, Pushover, etc. Be sure to also use the official capitalization for any abbreviations; e.g., IMDb has a lowercase 'b', whereas TMDB and TheTVDB have a capital 'B'.
- Title case headings, button text, and form labels. Note that verbs such as "is" should be capitalized, whereas prepositions like "from" should be lowercase (unless as the first or last word of the string, in which case they are also capitalized).
- Capitalize the first word in validation error messages, dropdowns, and form "tips." These strings should not end in punctuation.
- Ensure that toast notification strings are complete sentences ending in punctuation.
- If an additional description or "tip" is required for a form field, it should be styled using the global CSS class
label-tip
. - In full sentences, abbreviations like "info" or "auto" should not be used in place of full words, unless referencing the name/label of a specific setting or option which has an abbreviation in its name.
- Do your best to check for spelling errors and grammatical mistakes.
- Do not misspell "Maintainerr."
Attribution¶
This contribution guide was inspired by the Overseerr contribution guide.