We've been getting a lot of requests for plugins that simply don't fit in with Streamlink, don't have anywhere near enough details, or are generally not worth implementing. Up until now we've been dealing with some abuse from users in terms of what we will add/support due to not having it clearly documented what we will or will not support. This update to our documentation hopes to change that and clarify our stance for users. Fixes #1837
6.1 KiB
Contributing to Streamlink
Want to get involved? Thanks! There are plenty of ways to help!
Reporting issues
A bug is a demonstrable problem that is caused by the latest version of the code in the repository. Good bug reports are extremely helpful - thank you!
Please read the following guidelines before you report an issue:
-
See if the issue is already known — check the list of known issues.
-
Use the GitHub issue search — check if the issue has already been reported. If it has been, please comment on the existing issue.
-
Check if the issue has been fixed — the latest
master
or development branch may already contain a fix. -
Isolate the demonstrable problem — make sure that the code in the project's repository is definitely responsible for the issue.
-
Format your issue report — well formatted texts will make it much easier for developers to understand your report.
Please try to be as detailed as possible in your report too. What is your environment? What steps will reproduce the issue? What would you expect the outcome to be? All these details will help people to assess and fix any potential bugs. The issue template will aid you in structuring your report when submitting a new issue. If your report includes log output, please provide it as text in multi-line code blocks if possible for better readability. Thank you!
Feature requests
Feature requests are welcome, but take a moment to find out whether your idea fits with the scope and aims of the project. It's up to you to make a strong case to convince the project's developers of the merits of this feature. Please provide as much detail and context as possible.
Plugin requests
Plugin submissions and requests are a great way to improve Streamlink. Requests should be as detailed as possible and dedicated to only a single streaming service. Information about the service as well as explicit URLs for live streams are required, as are details regarding the website, what it is, who runs it, how it adds to Streamlink, etc. This allows for easier plugin review and implementation. We reserve the right to refuse to implement, develop, or update any plugin. In addition we may remove existing plugins at our own discretion.
Plugins which fall under the following categories will not be implemented or considered and the request will be closed:
-
Any kind of streaming service that uses DRM protection
-
Sites which are hosting stolen content as their main source of content
-
Sites which are primarily rehosting content that is available from a legitimate source (TV shows, sports, news, etc.)
-
Sites which require any sort of cable login or subscription
-
NSFW sites of a pornographic nature (cam sites, porn sites, etc.)
-
Sites which are mostly hosting VODs (stolen or legitimate), or require large changes to existing code to watch VODs
-
Sites where the maintainer has requested we not add their site to Streamlink
-
Sites which are unmaintained.
Pull requests
Good pull requests - patches, improvements, and new features - are a fantastic help. They should remain focused in scope and avoid containing unrelated commits.
Please ask first before embarking on any significant pull request (e.g. implementing features, refactoring code, porting to a different language), otherwise you risk spending a lot of time working on something that the project's developers might not want to merge into the project.
Please adhere to the coding conventions used throughout a project (indentation, white space, accurate comments, etc.) and any other requirements (such as test coverage).
Adhering to the following process is the best way to get your work included in the project:
-
Fork the project, clone your fork, and configure the remotes:
# Clone your fork of the repo into the current directory git clone git@github.com:<YOUR-USERNAME>/streamlink.git # Navigate to the newly cloned directory cd streamlink # Assign the original repo to a remote called "upstream" git remote add upstream https://github.com/streamlink/streamlink.git
-
If you cloned a while ago, get the latest changes from upstream
git checkout master git pull upstream master
-
Create a new topic branch (off the main project branch) to contain your feature, change, or fix:
git checkout -b <TOPIC-BRANCH-NAME>
-
Commit your changes in logical chunks. Please adhere to these git commit message guidelines or your code is unlikely be merged into the project. Use git's interactive rebase feature to tidy up your commits before making them public.
-
Locally merge (or rebase) the upstream branch into your topic branch:
git pull [--rebase] upstream master
-
Push your topic branch up to your fork:
git push origin <TOPIC-BRANCH-NAME>
-
Open a Pull Request with a clear title and description.
IMPORTANT: By submitting a patch, you agree to allow the project owners to license your work under the terms of the BSD 2-clause license.
Acknowledgements
This contributing guide has been adapted from HTML5 boilerplate's guide.