Skip to content

Commit

Permalink
Improve bump external releases tool to properly support palantir
Browse files Browse the repository at this point in the history
palantir-cli releases respect to all the other tools have a name that is more human like and less automation friendly.
Still, releases (also for prior tools) are based on tags that are automation friendly.

In here, we switch over from using the name of the release to use the name of the tag associated to the release
  • Loading branch information
macisamuele committed Jul 13, 2024
1 parent fb61cd8 commit 8c34eed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/bump_external_releases.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
def bump_release(github_project, tool_name):
try:
with urlopen(f"https://api.github.com/repos/{github_project}/releases/latest") as request: # nosec: disable=B310
latest_version = json.load(request)["name"].lstrip("v")
latest_version = json.load(request)["tag_name"].lstrip("v")
except: # noqa: E722 (allow usage of bare 'except')
traceback.print_exc()
return False
Expand Down Expand Up @@ -71,7 +71,7 @@ def bump_google_java_formatter():
def bump_palantir_java_formatter():
return bump_release(
github_project="jsonschema2dataclass/palantir-cli",
tool_name="palantir-cli",
tool_name="palantir",
)


Expand Down

0 comments on commit 8c34eed

Please sign in to comment.