-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RULES.md: initial document stating the LibreELEC repository usage rules. #4
Conversation
Would like to see a few more comments here from the Team, if we think it's ready let me know and I'll squash |
|
||
#### LibreELEC Testing Window | ||
|
||
Anything listed in the [Major Packages](https://github.com/LibreELEC/LibreELEC.tv/blob/master/RULES.md#major-packages) section **MUST** be submitted as a PR and reviewed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: You don't need https://github.com/LibreELEC/LibreELEC.tv/blob/master/RULES.md
on the target, so [Major Packages](#major-packages)
should work.
Other than the nits, it looks good. My only reservation is this line:
While I agree in general, what if we find a show-stopper bug in say, |
Sorry for offtopic comment, but I do not know where I can ask, your irc channel is invitation only. |
@WeinerRinkler - website and forums will be up shortly to explain, thanks :) |
Updated with @MilhouseVH changes. |
I would like the entire team to comment on this
This is just an initial document, it can be amended in the future to include any other rules that we need to define about the LibreELEC project.