From e7955f713c4a2d862ccd526206d0f7d4b262bc6d Mon Sep 17 00:00:00 2001 From: Stefan Zweifel Date: Sat, 3 May 2025 15:51:01 +0200 Subject: [PATCH] Emit warning if deprecated/removed options are used --- entrypoint.sh | 12 ++++++++++++ tests/git-auto-commit.bats | 19 +++++++++++++++++++ 2 files changed, 31 insertions(+) diff --git a/entrypoint.sh b/entrypoint.sh index b0f384b..1fa602d 100755 --- a/entrypoint.sh +++ b/entrypoint.sh @@ -27,6 +27,18 @@ _log() { } _main() { + if "$INPUT_SKIP_FETCH"; then + _log "warning" "skip_fetch has been removed in v6. It does not have any effect anymore."; + fi + + if "$INPUT_SKIP_CHECKOUT"; then + _log "warning" "skip_checkout has been removed in v6. It does not have any effect anymore."; + fi + + if "$INPUT_CREATE_BRANCH"; then + _log "warning" "create_branch has been removed in v6. It does not have any effect anymore."; + fi + _check_if_git_is_available _switch_to_repository diff --git a/tests/git-auto-commit.bats b/tests/git-auto-commit.bats index 254e561..432c4fe 100644 --- a/tests/git-auto-commit.bats +++ b/tests/git-auto-commit.bats @@ -38,6 +38,11 @@ setup() { export INPUT_DISABLE_GLOBBING=false export INPUT_INTERNAL_GIT_BINARY=git + # Deprecated variables. Will be removed in future versions + export INPUT_CREATE_BRANCH=false + export INPUT_SKIP_FETCH=false + export INPUT_SKIP_CHECKOUT=false + # Set GitHub environment variables used by the GitHub Action temp_github_output_file=$(mktemp -t github_output_test.XXXXX) export GITHUB_OUTPUT="${temp_github_output_file}" @@ -1161,3 +1166,17 @@ END run git tag assert_output "" } + +@test "it shows warning message if any deprecated options are used" { + INPUT_SKIP_FETCH=true + INPUT_SKIP_CHECKOUT=true + INPUT_CREATE_BRANCH=true + + run git_auto_commit + + assert_success + + assert_line "::warning::skip_fetch has been removed in v6. It does not have any effect anymore." + assert_line "::warning::skip_checkout has been removed in v6. It does not have any effect anymore." + assert_line "::warning::create_branch has been removed in v6. It does not have any effect anymore." +}