fix: Reverted changes for skip client cache (#37782)
## Description Added command for migration Fixes # https://app.zenhub.com/workspaces/stability-pod-6690c4814e31602e25cab7fd/issues/gh/appsmithorg/appsmith/37768 ## Automation /ok-to-test tags="" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > If you modify the content in this section, you are likely to disrupt the CI result for your PR. <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Enhanced the client build workflow for improved detection of modified files. - Streamlined conditions for executing build steps based on file changes. - Refined caching mechanisms for Yarn and build artifacts. - Improved handling of Git LFS commands in the build process. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
parent
62724e98f7
commit
94768bd47b
34
.github/workflows/client-build.yml
vendored
34
.github/workflows/client-build.yml
vendored
|
|
@ -66,23 +66,23 @@ jobs:
|
||||||
with:
|
with:
|
||||||
fetch-tags: true
|
fetch-tags: true
|
||||||
|
|
||||||
# - name: Get changed files in the client folder
|
- name: Get changed files in the client folder
|
||||||
# id: changed-files-specific
|
|
||||||
# uses: tj-actions/changed-files@v41
|
|
||||||
# with:
|
|
||||||
# files: "app/client/**"
|
|
||||||
|
|
||||||
- name: Updating the client changed file variable
|
|
||||||
id: changed-files-specific
|
id: changed-files-specific
|
||||||
run: echo "any_changed=true" >> "$GITHUB_OUTPUT"
|
uses: tj-actions/changed-files@v41
|
||||||
|
with:
|
||||||
|
files: "app/client/**"
|
||||||
|
|
||||||
#- name: Run step if any file(s) in the client folder change
|
# - name: Updating the client changed file variable
|
||||||
# if: steps.changed-files-specific.outputs.any_changed == 'true'
|
# id: changed-files-specific
|
||||||
# env:
|
# run: echo "any_changed=true" >> "$GITHUB_OUTPUT"
|
||||||
# ALL_CHANGED_FILES: ${{ steps.changed-files-specific.outputs.all_changed_files }}
|
|
||||||
# run: |
|
- name: Run step if any file(s) in the client folder change
|
||||||
# echo "One or more files in the server folder has changed."
|
if: steps.changed-files-specific.outputs.any_changed == 'true'
|
||||||
# echo "List all the files that have changed: $ALL_CHANGED_FILES"
|
env:
|
||||||
|
ALL_CHANGED_FILES: ${{ steps.changed-files-specific.outputs.all_changed_files }}
|
||||||
|
run: |
|
||||||
|
echo "One or more files in the server folder has changed."
|
||||||
|
echo "List all the files that have changed: $ALL_CHANGED_FILES"
|
||||||
|
|
||||||
- name: Check compliance
|
- name: Check compliance
|
||||||
if: inputs.pr != 0 && steps.changed-files-specific.outputs.any_changed == 'true'
|
if: inputs.pr != 0 && steps.changed-files-specific.outputs.any_changed == 'true'
|
||||||
|
|
@ -209,11 +209,11 @@ jobs:
|
||||||
git config --global user.email "$gituseremail"
|
git config --global user.email "$gituseremail"
|
||||||
git config --global user.name "$gituser"
|
git config --global user.name "$gituser"
|
||||||
git clone https://$cachetoken@github.com/appsmithorg/cibuildcache.git
|
git clone https://$cachetoken@github.com/appsmithorg/cibuildcache.git
|
||||||
git lfs install
|
|
||||||
git lfs migrate import --everything --yes
|
|
||||||
if [ "$reponame" = "appsmith" ]; then export repodir="CE"; fi
|
if [ "$reponame" = "appsmith" ]; then export repodir="CE"; fi
|
||||||
if [ "$reponame" = "appsmith-ee" ]; then export repodir="EE"; fi
|
if [ "$reponame" = "appsmith-ee" ]; then export repodir="EE"; fi
|
||||||
cd cibuildcache/$repodir/release/client
|
cd cibuildcache/$repodir/release/client
|
||||||
|
git lfs install
|
||||||
|
git lfs migrate import --everything --yes
|
||||||
git lfs pull ./build.tar
|
git lfs pull ./build.tar
|
||||||
mv ./build.tar ../../../../../build.tar
|
mv ./build.tar ../../../../../build.tar
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user