Not a huge issue but would it might be helpful in future debugging if we renamed the variable warc_file_path to meta_file_path to remove confusion about the warc_file (input file) and the meta_file (output file). Currently we use warc _file and then warc_file_path for different things.
Not a huge issue but would it might be helpful in future debugging if we renamed the variable
warc_file_pathtometa_file_pathto remove confusion about thewarc_file(input file) and themeta_file(output file). Currently we usewarc _fileand thenwarc_file_pathfor different things.https://github.com/unt-libraries/warc-metadata-sidecar/blob/bed3821d270764eec3fb3ac48c56ceac77ff06ae/warc_metadata_sidecar.py#L177