How to contribute code to Jellyfin
This page details how our repositories are organized, how to get started editing the code and creating your first pull request, and some general procedures around pull requests in Jellyfin.
What should you work on?
The best way to get going is to look through the Issues list, find an issue you would like to work on, and start hacking. Issues are triaged regularly by the administrative team, and labels assigned that should help you find issues within your skill-set. Once you start working on an issue, please comment on it stating your intent to work on the issue, to avoid unnecessary duplication of work.
Major Issue Types
A list of issue types can be found on the Issue guidelines page.
What if there isn't an issue?
If there isn't already an issue dealing with the changes you want to make, please create an issue to track it first, then ensure your PR(s) reference the issue in question. This is especially useful for bugs that are found and then fixed by the author, so both the original issue and the fix can be documented and tracked in detail.
How should you make changes?
Once you've found something you want to work on or improve, the next step is to make your changes in the code, test them, then submit a Pull Request (PR) on GitHub.
For simplicity, all examples assume the developer is operating on Linux with SSH access to GitHub, however the general ideas can be applied to HTTP-based GitHub interfaces, and can be translated to Windows or MacOS.
If you aren't familiar with Git, we recommend the official documentation to understand how this version control system works and how to use it.
Set up your copy of the repo
The first step is to set up a copy of the Git repository of the project you want to contribute to. Jellyfin follows a "fork, feature-branch, and PR" model for contributions.
On GitHub, "Fork" the Jellyfin repository you wish to contribute to, to your own user account using the "Fork" button in the relevant repository.
Clone your fork to your local machine and enter the directory:
git clone email@example.com:yourusername/projectname.git
Add the "upstream" remote, which allows you to pull down changes from the main project easily:
git remote add upstream firstname.lastname@example.org:jellyfin/projectname.git
Initialize the Git submodules; most projects have at least one:
git submodule update --init
Avoid editing files inside of submodules. Doing so may result in unintended side effects, including overwriting of your work. Always modify the submodule in its own cloned repository elsewhere.
Build the Jellyfin Web project with Yarn or NPM, and copy the location of the resulting
distfolder. In your
Jellyfin.Serverproject add an environment variable named
JELLYFIN_WEB_DIRwith the value set to the full path of your
You will now be ready to begin building or modifying the project.
Make changes to the repo
Once you have your repository, you can get to work.
Rebase your local branches against upstream
masterso you are working off the latest changes:
git fetch --all
git rebase upstream/master
Create a local feature branch off of
masterto make your changes:
git checkout -b my-feature master
Make your changes and commits to this local feature branch.
Repeat step 1 on your local feature branch once you're done your work, to ensure you have no conflicts with other work done since you stated.
Push up your local feature branch to your GitHub fork:
git push --set-upstream origin my-feature
On GitHub, create a new PR against the upstream
masterbranch following the advice below.
Once your PR is merged, ensure you keep your local branches up-to-date:
git fetch --all
git checkout master
git rebase upstream/master
git push -u origin master
Delete your local feature branch if you no longer need it:
git branch -d my-feature
If it's your first time contributing code to a particular repository, please add yourself to the
CONTRIBUTORS.md file at the bottom of the
Jellyfin Contributors section. While GitHub does track this, having the written document makes things clearer if the code leaves GitHub and lets everyone quickly see who's worked on the project for copyright or praise!
From time to time, major projects may come up that require multiple PRs and contributions from multiple people. For these tasks, feature branches specific to the feature should be created, based off of
master. This helps allow the work to progress without breaking
master for long periods, and allowing those interested in that particular project the ability to work at their own pace instead of racing to fix a broken feature before the next release. To create a feature branch, please communicate with a Core team member and that can be arranged.
Once the feature a feature branch was created for is ready, it can be merged in one shot into
master and the feature branch removed. Alternatively, for very-long-lived features, certain "stable" snapshots can be merged into
master as required.
The Master Branch
master branch is the primary face of the project and main development branch. Except for emergency release hotfixes, all PRs should target
master. As a general rule, no PR should break master and all PRs should be tested before merging to ensure this does not occur. We're only human and this is still likely to happen, but you should generally be safe to build off of
master if you want the latest and greatest version of Jellyfin.
Testing a Pull Request
To test someone else's pull request, you must import the changes to your local repository.
- Fetch the changes in a pull request and link them to a new local branch:
git fetch upstream pull/<PR_ID>/head:my-testing-branch
<PR_ID> is pull request number on GitHub.
Checkout the new local branch:
git checkout my-testing-branch
Perform any testing or build required to test, then return to master and delete the branch:
git checkout master
git branch -D my-testing-branch
Pull Request Guidelines
When submitting a new PR, please ensure you do the following things. If you haven't, please read How to Write a Git Commit Message as it is a great resource for writing useful commit messages.
Before submitting a PR, squash "junk" commits together to keep the overall history clean. A single commit should cover a single significant change: avoid squashing all your changes together, especially for large PRs that touch many files, but also don't leave "fixed this", "whoops typo" commits in your branch history as this is needless clutter in the final history of the project.
Write a good title that quickly describes what has been changed. For example, "Add LDAP support to Jellyfin". As mentioned in How to Write a Git Commit Message, always use the imperative mood, and keep the title short but descriptive. The title will eventually be a changelog entry, so please try to use proper capitalization but no punctuation; note that the Core team may alter titles to better confirm to this standard before merging.
For anything but the most trivial changes that can be described fully in the (short) title, follow the PR template and write a PR body to describe, in as much detail as possible:
Why the changes are being made. Reference specific issues with keywords (
addresses, etc.) if at all possible.
How you approached the issue (if applicable) and briefly describe the changes, especially for large PRs.
If your pull request isn't finished yet please mark it as a "draft" when you open it. While this tag is in place, the pull request won't be merged, and reviews should remain as comments only. Once you're happy with the final state of your PR, please remove this tag; forgetting to do so might result in your PR being unintentionally ignored as still under active development! Inactive WIPs may occasionally elicit pings from the team inquiring on the status, and closed if there is no response.
Avoid rebasing and force-pushing to large or complex pull requests if at all possible, and especially after reviews. It forces unnecessary reviews to verify the changes are still okay and build properly.
Expect review and discussion. If you can't back up your changes with a good description and through review, please reconsider whether it should be done at all. All PRs to
devrequire at least one approving review from an administrative team member, however we welcome and encourage reviews from any contributor, especially if it's in an area you are knowledgeable about. More eyes are always better.
All PRs require review by at least two team members before being merged into
master, though reviews from any contributor are welcome! After the second team member review the PR may be merged immediately, or more review or feedback requested explicitly from other contributors if required.
Building and Testing Inside a Docker Container
We need to install all development dependencies and pull down the code inside the container before we can compile and run.
Run each commmand on a separate line. The container we'll test in is named
jftest. Within Docker, anytime the entrypoint executable is terminated, the session restarts, so just exec into it again to continue. This is also why we explicitly kill it to reload the new version.
docker exec -ti jftest bash apt-get update && apt-get install git gnupg wget apt-transport-https curl wget -qO- https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.asc.gpg && mv microsoft.asc.gpg /etc/apt/trusted.gpg.d/ wget -q https://packages.microsoft.com/config/debian/10/prod.list && mv prod.list /etc/apt/sources.list.d/microsoft-prod.list curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - && echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list apt-get update && apt-get install dotnet-sdk-3.1 yarn cd /opt && git clone https://github.com/jellyfin/jellyfin.git && git clone https://github.com/jellyfin/jellyfin-web.git mv /jellyfin/ /jellyfin.bak && cd /opt/jellyfin && dotnet publish Jellyfin.Server --configuration Debug --output="/jellyfin" --self-contained --runtime linux-x64 cd /opt/jellyfin-web && yarn install && yarn build && cp -r /opt/jellyfin-web/dist /jellyfin/jellyfin-web kill -15 6
First, complete the steps above to setup your container to build the master branch.
<PR_ID> is pull request number on GitHub.
docker exec -ti jftest bash cd /opt/jellyfin git fetch origin pull/<PR_ID>/head:my-testing-branch git merge my-testing-branch dotnet publish Jellyfin.Server --configuration Debug --output="/jellyfin" --self-contained --runtime linux-x64 kill -15 6