Skip to content

Commit

Permalink
ci: Making minor improvements to the CI workflow files (#33636)
Browse files Browse the repository at this point in the history
This is trivial housekeeping task to keep the workflow files consistent
and remove warnings.
  • Loading branch information
mohanarpit authored May 23, 2024
1 parent 9911049 commit 9faab94
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-test-limited.yml
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ jobs:
# In case this is second attempt try restoring status of the prior attempt from cache
- name: Restore the previous run result
id: cache-appsmith
uses: martijnhols/actions-cache@v3.0.2
uses: actions/cache/restore@v4
with:
path: |
~/run_result
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/client-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ jobs:
with:
path: app/client/.yarn/cache
key: v1-yarn3-${{ hashFiles('app/client/yarn.lock') }}
restore-keys: |
v1-yarn3-

# Set status = success
- name: Save the status of the run
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/client-prettier.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ jobs:
# Restoring the cache manually instead
- name: Restore Yarn cache
if: steps.run_result.outputs.run_result != 'success'
uses: actions/cache@v4
uses: actions/cache/restore@v4
with:
path: app/client/.yarn/cache
key: v1-yarn3-${{ hashFiles('app/client/yarn.lock') }}
Expand All @@ -78,4 +78,3 @@ jobs:
with:
path: app/client/.yarn/cache
key: v1-yarn3-${{ hashFiles('app/client/yarn.lock') }}
restore-keys: v1-yarn3-
2 changes: 1 addition & 1 deletion .github/workflows/pr-cypress.yml
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ jobs:
path: ~/combined_failed_spec_ci
overwrite: true

- name: Get Latest flaky Tests
- name: Get latest flaky tests
shell: bash
run: |
curl --request POST --url https://yatin-s-workspace-jk8ru5.us-east-1.xata.sh/db/CypressKnownFailures:main/tables/CypressKnownFailuires/query --header 'Authorization: Bearer ${{ secrets.XATA_TOKEN }}' --header 'Content-Type: application/json'|jq -r |grep Spec|cut -d ':' -f 2 2> /dev/null|sed 's/"//g'|sed 's/,//g' > ~/knownfailures
Expand Down
9 changes: 5 additions & 4 deletions .github/workflows/test-build-docker-image.yml
Original file line number Diff line number Diff line change
Expand Up @@ -153,15 +153,16 @@ jobs:
rm -f ~/combined_failed_spec_ci
# Download failed_spec_ci list for all CI container jobs
- uses: actions/download-artifact@v4
- name: Download failed_spec_ci list for all CI container jobs
uses: actions/download-artifact@v4
if: needs.ci-test.result != 'success'
id: download_ci
with:
name: failed-spec-ci-${{github.run_attempt}}
path: ~/failed_spec_ci

# In case for any ci job failure, create combined failed spec
- name: "combine all specs for CI"
- name: Combine all specs for CI
if: needs.ci-test.result != 'success'
run: |
rm -f ~/combined_failed_spec_ci
Expand Down Expand Up @@ -200,7 +201,7 @@ jobs:
echo "slack_icon=:warning:" >> $GITHUB_OUTPUT
fi
- name: Slack Notification
- name: Slack notification
continue-on-error: true
if: always()
uses: rtCamp/action-slack-notify@v2
Expand Down Expand Up @@ -253,7 +254,7 @@ jobs:

# Upload combined failed CI spec list to a file
# This is done for debugging.
- name: upload combined failed spec
- name: Upload combined failed spec
if: needs.ci-test.result != 'success'
uses: actions/upload-artifact@v4
with:
Expand Down

0 comments on commit 9faab94

Please sign in to comment.