search

web-platform-tests / wpt.fyi

web-platform-tests dashboard
https://wpt.fyi/
Other
182 stars 88 forks source link

[results-processor] Refer to download an archive file instead of "azure URL" #3958

Open gsnedders opened 3 weeks ago

gsnedders commented 3 weeks ago

Currently we have function signatures like:

def download(self, results, screenshots, azure_url): pass

Instead of explicitly calling it an "azure URL" (which isn't justified in the results-processor — there's nothing Azure-specific about it), we should just have some concept of an "archive URL" pointing at an archive.

(For #3906 I'm just going to pass azure_url [sic] to the processor, rather than doing this renaming, but we should keep this on the backlog.)

past commented 1 week ago

Eventually you did end up renaming it to archive_url right? Is there anything more to do here?