2013-04-09, 22:38
As already said before, I don't think github is a good idea for all devs.
People should be able to use what they want: svn, mercurial, or even just zip if they want.
I think a website is a good idea. People could give an url (git/svn/hg/zip) like they do today via e-mail.
With a basic API, we should even be able to provide a small script to make the pull request, so that more advanced users don't have to fill a web form.
In the background, a script could do basic checks and make a commit in a dev repo.
In case of obvious mistakes (invalid xbmc.python version, eol...) the pull request would be refused and the author can fix it without having to wait for team XBMC feedback.
If it passes the basic checks, it will wait for review by a team XBMC member.
This is basically what I do today with my script to process e-mails pull requests (https://github.com/beenje/addon-pr):
- get the add-on
- basic checks
- commit to my local repo if it passes the checks
Then:
- human review
- push to repo if everything ok or delete commit if not
So basically it's "just" to put that script behind a web server ;-)
People should be able to use what they want: svn, mercurial, or even just zip if they want.
I think a website is a good idea. People could give an url (git/svn/hg/zip) like they do today via e-mail.
With a basic API, we should even be able to provide a small script to make the pull request, so that more advanced users don't have to fill a web form.
In the background, a script could do basic checks and make a commit in a dev repo.
In case of obvious mistakes (invalid xbmc.python version, eol...) the pull request would be refused and the author can fix it without having to wait for team XBMC feedback.
If it passes the basic checks, it will wait for review by a team XBMC member.
This is basically what I do today with my script to process e-mails pull requests (https://github.com/beenje/addon-pr):
- get the add-on
- basic checks
- commit to my local repo if it passes the checks
Then:
- human review
- push to repo if everything ok or delete commit if not
So basically it's "just" to put that script behind a web server ;-)