Use pathlib.Path everywhere
It's imported, let's use it more! Signed-off-by: Nils VAN ZUIJLEN <nils.van-zuijlen@mailo.com>
This commit is contained in:
parent
7b07514b93
commit
a6805cb244
1 changed files with 9 additions and 10 deletions
|
@ -10,13 +10,13 @@ from jinja2 import Environment, FileSystemLoader
|
||||||
|
|
||||||
def generate_READMEs(app_path):
|
def generate_READMEs(app_path):
|
||||||
|
|
||||||
if not os.path.exists(app_path):
|
if not app_path.exists():
|
||||||
raise Exception("App path provided doesn't exists ?!")
|
raise Exception("App path provided doesn't exists ?!")
|
||||||
|
|
||||||
manifest = json.load(open(os.path.join(app_path, "manifest.json")))
|
manifest = json.load(open(app_path / "manifest.json"))
|
||||||
upstream = manifest.get("upstream", {})
|
upstream = manifest.get("upstream", {})
|
||||||
|
|
||||||
if not upstream and not os.path.exists(os.path.join(app_path, "doc", "DISCLAIMER.md")):
|
if not upstream and not (app_path / "doc" / "DISCLAIMER.md").exists():
|
||||||
print("There's no 'upstream' key in the manifest, and doc/DISCLAIMER.md doesn't exists - therefore assuming that we shall not auto-update the README.md for this app yet.")
|
print("There's no 'upstream' key in the manifest, and doc/DISCLAIMER.md doesn't exists - therefore assuming that we shall not auto-update the README.md for this app yet.")
|
||||||
return
|
return
|
||||||
|
|
||||||
|
@ -26,24 +26,23 @@ def generate_READMEs(app_path):
|
||||||
|
|
||||||
template = env.get_template(f'README{lang_suffix}.md.j2')
|
template = env.get_template(f'README{lang_suffix}.md.j2')
|
||||||
|
|
||||||
if os.path.exists(os.path.join(app_path, "doc", "screenshots")):
|
if (app_path / "doc" / "screenshots").exists():
|
||||||
screenshots = os.listdir(os.path.join(app_path, "doc", "screenshots"))
|
screenshots = os.listdir(os.path.join(app_path, "doc", "screenshots"))
|
||||||
if ".gitkeep" in screenshots:
|
if ".gitkeep" in screenshots:
|
||||||
screenshots.remove(".gitkeep")
|
screenshots.remove(".gitkeep")
|
||||||
else:
|
else:
|
||||||
screenshots = []
|
screenshots = []
|
||||||
|
|
||||||
if os.path.exists(os.path.join(app_path, "doc", f"DISCLAIMER{lang_suffix}.md")):
|
if (app_path / "doc" / f"DISCLAIMER{lang_suffix}.md").exists():
|
||||||
disclaimer = open(os.path.join(app_path, "doc", f"DISCLAIMER{lang_suffix}.md")).read()
|
disclaimer = (app_path / "doc" / f"DISCLAIMER{lang_suffix}.md").read_text()
|
||||||
# Fallback to english if maintainer too lazy to translate the disclaimer idk
|
# Fallback to english if maintainer too lazy to translate the disclaimer idk
|
||||||
elif os.path.exists(os.path.join(app_path, "doc", "DISCLAIMER.md")):
|
elif (app_path / "doc" / "DISCLAIMER.md").exists():
|
||||||
disclaimer = open(os.path.join(app_path, "doc", "DISCLAIMER.md")).read()
|
disclaimer = (app_path / "doc" / "DISCLAIMER.md").read_text()
|
||||||
else:
|
else:
|
||||||
disclaimer = None
|
disclaimer = None
|
||||||
|
|
||||||
out = template.render(lang=lang, upstream=upstream, screenshots=screenshots, disclaimer=disclaimer, manifest=manifest)
|
out = template.render(lang=lang, upstream=upstream, screenshots=screenshots, disclaimer=disclaimer, manifest=manifest)
|
||||||
with open(os.path.join(app_path, f"README{lang_suffix}.md"), "w") as f:
|
(app_path / f"README{lang_suffix}.md").write_text(out)
|
||||||
f.write(out)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == "__main__":
|
if __name__ == "__main__":
|
||||||
|
|
Loading…
Reference in a new issue