Carlos Paradis
Carlos Paradis
@beydlern just noticed the review request. Is this good to go on review?
@ian-lastname Please add here the notes requested during the last meeting Friday: * Screenshots / urls / examples of how the pipermail .txt file can be obtained to manually prompt...
@ian-lastname If the code already exists, could you make a commit to just place it back? I have not start reviewing your code yet
The pipermail mbox refresher has a main IF and ELSE. In the case the IF enters, it will default the entire code logic to download_pipermail. Download pipermail downloads the main...
download_mod_mbox was not tested on a project that the data was not available to current date, as most apache projects had them. I suspect there will be a problem where...
@daomcgill You can use that or any of the ones here: https://mta.openssl.org/mailman/listinfo/ The behavior of going backwards is not intended. Neither is making 4 calls to download the same file:...
@daomcgill, on the closing week of this PR, I went over what had to be fixed for this to be merged. The summary of that can be found in this...
@daomcgill > If you get a chance, would you mind checking my updated specifications? I want to make sure this part is right before I do anything else. Ian's specification...
Ian's format of specification is also generally what you want: The function signature and a few bullets giving me some idea of your logic under said function. Try to do...
> convert_pipermail_to_mbox(filelist): I don't believe you need this function. Just try to save the files as .mbox instead of .txt when naming them and see if parse_mbox() recognizes it. >...