Skip to content

Commit

Permalink
Add cases for packages that do not have JS or Python
Browse files Browse the repository at this point in the history
- Just ignore them in that case
- Log an error if a package is unrecognized
  • Loading branch information
mofojed committed Mar 18, 2024
1 parent 2668989 commit c98c669
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions tools/update_version.sh
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,9 @@ case "$package" in
packaging)
update_file packaging/setup.cfg 'version = ' '' "$extra"
;;
auth-keycloak | dashboard-object-viewer | table-example)
# Packages that don't have any Python to publish, just ignore
;;
*)
{
log_error "Unhandled plugin $package. You will need to add wiring in $SCRIPT_NAME"
Expand All @@ -149,6 +152,14 @@ case "$package" in
# The working directory is already `plugins/<package-name>`, so we just specify workspace as `src/js` and it does the right thing
npm version "$npm_version" --workspace=src/js
;;
json | packaging | utilities)
# Packages that don't have any JS to publish, just ignore
;;
*)
{
log_error "Unhandled JS plugin $package. You will need to add JS wiring in $SCRIPT_NAME"
exit 90
}
esac

log_info "Done updating $package version to $version${extra}"

0 comments on commit c98c669

Please sign in to comment.