LGTM now. :)

Looks good to me, thanks ! :)

I updated the code as you suggested (I had to raise exceptions in some cases otherwise my editor was complaining with No return statement).

refactor the loader code

add myself to contributors

add explanation about opam root

use stats for tests, remove useless comments and print statements

fix opam loader task so the loader does use some storage

use exception rather than print + exit, use more explicit messages

revision info check works

try to test snapshot info

use tmpdir, use octal notation for permissions

fix task test, check that the opam root is valid

cut long lines, add task test

I've added some tests and fixed one of the review (I don't think the other one is easily do-able for now...).

set message to version

add tests

the loader works :-)

fix typo

complete loader implementation

fix bug

add extra_loader_arguments

fix copyright years

add missing copyright header

more explanations about opam in the docstring

Please don't merge now. I noticed while writing the loader that I'll need to add extra arguments to the loader. Will update the diff soon.

remove comment

add tasks tests

add missing __init__.py in test to fix build

I don't understand the build failure, does someone have any clue ?

add tests

@vlorentz thanks but it won't be necessary. I was on vacation, I'll update the diff this week with the tests. :)

rebase and fix build

update the code according to the reviews

update the code according to the reviews