In general, I think it's good to use full-length variable names in for loops like this, so like, `for link in source_links.keys()`. That way, the variable's purpose is more obvious in the block to come. This one jumped out at me in particular because a monospace `l` is so similar to a `1`, and it's used as index.