Skip to content

Commit 8057ca3

Browse files
chore: resolving review comments of dev to main pr (398) (#415)
### Description Resolving review comments of PR: #398 - This PR addresses [this comment](#398 (comment)) by removing redundant updates to both test-inventory and setup-workflow for docs-only changes. Updating both is unnecessary since failure in either prevents tests from running. - Test runs: https://github.com/splunk/splunk-add-on-for-amazon-web-services/actions/runs/15754544427/job/44407048288?pr=1486 - A few minor review comment changes were also made; they don't require testing. ### Checklist - [ ] `README.md` has been updated or is not required - [ ] push trigger tests - [ ] manual release test - [ ] automated releases test - [ ] pull request trigger tests - [ ] schedule trigger tests - [ ] workflow errors/warnings reviewed and addressed ### Testing done (for each selected checkbox, the corresponding test results link should be listed here)
1 parent 53f181c commit 8057ca3

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

.github/workflows/reusable-build-test-release.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ on:
3939
required: false
4040
description: "branch for k8s manifests to run the tests on"
4141
type: string
42-
default: "v3.4.1"
42+
default: "v3.5.0"
4343
scripted-inputs-os-list:
4444
required: false
4545
description: "list of OS used for scripted input tests"
@@ -139,7 +139,6 @@ jobs:
139139
set -o xtrace
140140
# List all files modified in the commit or PR
141141
changed_files=$(git diff --name-only --diff-filter=ACMRT ${{ github.event.pull_request.base.sha }} ${{ github.sha }})
142-
# changed_files=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }})
143142
144143
# Check if any of the changed files are not in docs/*, mkdocs.yml, or .github/workflows/docs.yml
145144
if echo "$changed_files" | grep -vqE '^(docs/|mkdocs.yml|.github/workflows/docs.yml)'; then
@@ -198,9 +197,7 @@ jobs:
198197
case "${{ github.event_name }}" in
199198
"pull_request")
200199
labels=$(echo '${{ toJSON(github.event.pull_request.labels) }}' | jq -r '.[] | .name')
201-
if ${{ needs.check-docs-changes.outputs.docs-only == 'true' }}; then
202-
echo -e "\033[1;33m⚠️ Skipping tests as there are only documentation related changes ⚠️\033[0m"
203-
elif ${{ github.base_ref == 'main' }} && ${{ contains(github.event.pull_request.labels.*.name, 'use_labels') }}; then
200+
if ${{ github.base_ref == 'main' }} && ${{ contains(github.event.pull_request.labels.*.name, 'use_labels') }}; then
204201
for test_type in "${TESTSET[@]}"; do
205202
if [[ "$labels" =~ $test_type ]]; then
206203
EXECUTE_LABELED["$test_type"]="true"

runbooks/update_appinspect_cli_action.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Once Splunk AppInspect team releases AppInspect CLI - we need to make sure that
2222

2323
- create a PR in this repository with a new version of the action ([example PR](https://github.com/splunk/addonfactory-workflow-addon-release/pull/247))
2424
- make sure that PR is towards `main` branch
25-
- make sure the tile of the PR follows the format: "fix: update AppInspect CLI action to v.X.Y"
25+
- make sure the title of the PR follows the format: "fix: update AppInspect CLI action to v.X.Y"
2626
- make sure that the pipeline is green
2727
- attach a link to a test run of reusable workflow
2828
- get review from the team

0 commit comments

Comments
 (0)