commit | 97520c004073497833fcad6cc8ccca2a66bc667f | [log] [tgz] |
---|---|---|
author | Dimitri Papadopoulos <3234522+DimitriPapadopoulos@users.noreply.github.com> | Fri Sep 10 15:49:14 2021 +0200 |
committer | Dimitri Papadopoulos <3234522+DimitriPapadopoulos@users.noreply.github.com> | Fri Sep 10 15:54:34 2021 +0200 |
tree | f8b8848fd4c444b2fa046ae99078adf716282cb3 | |
parent | f3901ed018c401a85a570b7c56cd6d1c1d3e7575 [diff] |
Fix LGTM recommendation: Module is imported with 'import' and 'import from' Module 'os' is imported with both 'import' and 'import from' I feel we don't need this abbreviation here: - Lines are not too long. - os.path is known by all Python developpers, while the op abbreviation is not (I believe the osp abbreviation is more common). And actually we had both os.path.exists() and op.join() in the code. That's confusing.