Page MenuHomeSoftware Heritage

aleo (Léo Andrès)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 30 2021, 11:58 AM (30 w, 16 h)

Recent Activity

Thu, Nov 4

aleo accepted D6606: opam: Write package definitions to the extrinsic metadata storage.

LGTM now. :)

Thu, Nov 4, 7:53 PM

Jul 20 2021

aleo accepted D6009: opam: Avoid unnecessary raise instructions.

Looks good to me, thanks ! :)

Jul 20 2021, 4:20 PM
aleo added a comment to D5975: add opam loader.

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

Jul 20 2021, 2:15 PM
aleo updated the diff for D5975: add opam loader.

refactor the loader code

Jul 20 2021, 2:15 PM

Jul 19 2021

aleo added inline comments to D5975: add opam loader.
Jul 19 2021, 6:00 PM
aleo updated the diff for D5975: add opam loader.

add myself to contributors

Jul 19 2021, 5:57 PM
aleo updated the diff for D5975: add opam loader.

add explanation about opam root

Jul 19 2021, 5:56 PM
aleo added inline comments to D5975: add opam loader.
Jul 19 2021, 3:05 PM
aleo updated the diff for D5975: add opam loader.

use stats for tests, remove useless comments and print statements

Jul 19 2021, 3:05 PM
aleo updated the diff for D5975: add opam loader.

fix opam loader task so the loader does use some storage

Jul 19 2021, 2:17 PM

Jul 13 2021

aleo added inline comments to D5975: add opam loader.
Jul 13 2021, 4:57 PM
aleo updated the diff for D5975: add opam loader.

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

Jul 13 2021, 4:54 PM
aleo updated the diff for D5975: add opam loader.

revision info check works

Jul 13 2021, 4:33 PM
aleo updated the diff for D5975: add opam loader.

try to test snapshot info

Jul 13 2021, 4:20 PM
aleo updated the diff for D5975: add opam loader.

use tmpdir, use octal notation for permissions

Jul 13 2021, 2:54 PM
aleo updated the diff for D5975: add opam loader.

fix task test, check that the opam root is valid

Jul 13 2021, 12:12 PM
aleo updated the diff for D5975: add opam loader.

cut long lines, add task test

Jul 13 2021, 11:12 AM

Jul 12 2021

aleo added a comment to D5975: add opam loader.

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

Jul 12 2021, 6:21 PM
aleo updated the diff for D5975: add opam loader.

set message to version

Jul 12 2021, 6:17 PM
aleo updated the diff for D5975: add opam loader.

add tests

Jul 12 2021, 6:00 PM

Jul 8 2021

aleo updated the diff for D5975: add opam loader.

the loader works :-)

Jul 8 2021, 3:12 PM

Jul 7 2021

aleo updated the diff for D5975: add opam loader.

fix typo

Jul 7 2021, 9:30 PM
aleo updated the diff for D5975: add opam loader.

complete loader implementation

Jul 7 2021, 5:57 PM

Jul 6 2021

aleo updated the diff for D5975: add opam loader.

fix bug

Jul 6 2021, 6:44 PM
aleo requested review of D5975: add opam loader.
Jul 6 2021, 6:40 PM
aleo updated the diff for D5808: add opam lister.

add extra_loader_arguments

Jul 6 2021, 3:19 PM

Jul 5 2021

aleo updated the diff for D5808: add opam lister.

fix copyright years

Jul 5 2021, 7:08 PM
aleo updated the diff for D5808: add opam lister.

add missing copyright header

Jul 5 2021, 7:03 PM
aleo updated the diff for D5808: add opam lister.

more explanations about opam in the docstring

Jul 5 2021, 6:58 PM
aleo planned changes to D5808: add opam lister.

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.

Jul 5 2021, 5:54 PM
aleo updated the diff for D5808: add opam lister.

remove comment

Jul 5 2021, 5:50 PM
aleo updated the diff for D5808: add opam lister.

add tasks tests

Jul 5 2021, 5:49 PM
aleo added inline comments to D5808: add opam lister.
Jul 5 2021, 5:35 PM

Jun 28 2021

aleo updated the diff for D5808: add opam lister.

add missing __init__.py in test to fix build

Jun 28 2021, 2:46 PM

Jun 23 2021

aleo added a comment to D5808: add opam lister.

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

Jun 23 2021, 5:39 PM
aleo updated the diff for D5808: add opam lister.

add tests

Jun 23 2021, 5:26 PM
aleo added a comment to D5808: add opam lister.

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

Jun 23 2021, 1:45 PM

Jun 9 2021

aleo accepted D5838: Add opam to the common dockerfile, for swh.lister.
Jun 9 2021, 3:36 PM
aleo updated the diff for D5808: add opam lister.

rebase and fix build

Jun 9 2021, 3:15 PM
aleo updated the diff for D5808: add opam lister.

update the code according to the reviews

Jun 9 2021, 12:50 PM

Jun 8 2021

aleo updated the diff for D5808: add opam lister.

update the code according to the reviews

Jun 8 2021, 3:22 PM

Jun 3 2021

aleo requested review of D5808: add opam lister.
Jun 3 2021, 12:19 PM