Skip to content

Commit

Permalink
Remove deprecated charts-repo-url option (#123)
Browse files Browse the repository at this point in the history
* Remove deprecated charts-repo-url option

Signed-off-by: James Tanner-McLeod <james.tanner@tanndev.com>

* empty commit to trigger ci

Signed-off-by: David J. M. Karlsen <david@davidkarlsen.com>

Signed-off-by: James Tanner-McLeod <james.tanner@tanndev.com>
Signed-off-by: David J. M. Karlsen <david@davidkarlsen.com>
Co-authored-by: David J. M. Karlsen <david@davidkarlsen.com>
  • Loading branch information
jftanner and davidkarlsen committed Nov 30, 2022
1 parent 38cfeac commit 89fff88
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 24 deletions.
7 changes: 0 additions & 7 deletions action.yml
Expand Up @@ -16,9 +16,6 @@ inputs:
description: The charts directory
required: false
default: charts
charts_repo_url:
description: "The GitHub Pages URL to the charts repo (default: https://<owner>.github.io/<repo>)"
required: false
install_dir:
description: "Where to install the cr tool"
required: false
Expand Down Expand Up @@ -47,10 +44,6 @@ runs:
args+=(--config "${{ inputs.config }}")
fi
if [[ -n "${{ inputs.charts_repo_url }}" ]]; then
args+=(--charts-repo-url "${{ inputs.charts_repo_url }}")
fi
if [[ -z "${{ inputs.install_dir }}" ]]; then
install="$RUNNER_TOOL_CACHE/cr/${{ inputs.version }}/$(uname -m)"
echo "$install" >> "$GITHUB_PATH"
Expand Down
18 changes: 1 addition & 17 deletions cr.sh
Expand Up @@ -28,7 +28,6 @@ Usage: $(basename "$0") <options>
-v, --version The chart-releaser version to use (default: $DEFAULT_CHART_RELEASER_VERSION)"
--config The path to the chart-releaser config file
-d, --charts-dir The charts directory (default: charts)
-u, --charts-repo-url The GitHub Pages URL to the charts repo (default: https://<owner>.github.io/<repo>)
-o, --owner The repo owner
-r, --repo The repo name
-n, --install-dir The Path to install the cr tool
Expand All @@ -43,7 +42,6 @@ main() {
local charts_dir=charts
local owner=
local repo=
local charts_repo_url=
local install_dir=
local install_only=
local skip_packaging=
Expand Down Expand Up @@ -135,16 +133,6 @@ parse_command_line() {
exit 1
fi
;;
-u|--charts-repo-url)
if [[ -n "${2:-}" ]]; then
charts_repo_url="$2"
shift
else
echo "ERROR: '-u|--charts-repo-url' cannot be empty." >&2
show_help
exit 1
fi
;;
-o|--owner)
if [[ -n "${2:-}" ]]; then
owner="$2"
Expand Down Expand Up @@ -203,10 +191,6 @@ parse_command_line() {
exit 1
fi

if [[ -z "$charts_repo_url" ]]; then
charts_repo_url="https://$owner.github.io/$repo"
fi

if [[ -z "$install_dir" ]]; then
local arch
arch=$(uname -m)
Expand Down Expand Up @@ -294,7 +278,7 @@ release_charts() {
}

update_index() {
local args=(-o "$owner" -r "$repo" -c "$charts_repo_url" --push)
local args=(-o "$owner" -r "$repo" --push)
if [[ -n "$config" ]]; then
args+=(--config "$config")
fi
Expand Down

0 comments on commit 89fff88

Please sign in to comment.