Skip to content

Commit

Permalink
CI: Bump dependency
Browse files Browse the repository at this point in the history
Initially, Arm Cortex-A port checks were conducted using 'pwsh' (Windows
PowerShell) on a Windows host. To maintain consistency, the continuous
integration pipeline shifted to a Linux host. Consequently, this change
eliminated the use of Windows PowerShell, which effectively disabled the
Arm Cortex-A port checks.
  • Loading branch information
jserv committed Nov 21, 2024
1 parent ca456a4 commit 473f3ae
Show file tree
Hide file tree
Showing 5 changed files with 22 additions and 47 deletions.
38 changes: 6 additions & 32 deletions .github/workflows/ports_arch_check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@
name: ports_arch_check

# Controls when the action will run. Triggers the workflow on push or pull request
# events but only for the master branch
# events but only for the dev branch
on:
pull_request:
branches: [ master ]
branches: [ dev ]
paths:
- ".github/workflows/ports_arch_check.yml"
- 'common/**'
Expand All @@ -28,7 +28,7 @@ jobs:
steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
- name: Checkout sources recursively
uses: actions/checkout@v2
uses: actions/checkout@v4
with:
token: ${{ secrets.REPO_SCOPED_TOKEN }}
submodules: true
Expand All @@ -38,36 +38,10 @@ jobs:
run: |
scripts/copy_armv7_m.sh && scripts/copy_armv8_m.sh && scripts/copy_module_armv7_m.sh
if [[ -n $(git status --porcelain -uno) ]]; then
echo "Ports for ARM architecture is not updated"
echo "Ports for Arm architecture is not updated"
git status
exit 1
fi
cortex-a:
# Check ports for cortex-a
runs-on: windows-latest

# Steps represent a sequence of tasks that will be executed as part of the job
steps:
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
- name: Checkout sources recursively
uses: actions/checkout@v2
with:
token: ${{ secrets.REPO_SCOPED_TOKEN }}
submodules: true

# Copy ports arch
- name: Copy ports arch
run: |
cd ports_arch/ARMv7-A
pwsh -Command ./update.ps1 -PortSets tx -CopyCommonFiles -CopyPortFiles -CopyExample -PatchFiles
cd ../../ports_arch/ARMv8-A
pwsh -Command ./update.ps1 -PortSets tx,tx_smp -CopyCommonFiles -CopyPortFiles -CopyExample -PatchFiles
if ((git status --porcelain -uno) -ne $null) {
Write-Host "Ports for ARM architecture is not updated"
git status
Exit 1
}
# FIXME: Re-enable the checks for Arm Cortex-A
# cortex-a:
23 changes: 12 additions & 11 deletions .github/workflows/regression_template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ jobs:
run: ${{ inputs.test_script }}

- name: Publish Test Results
uses: EnricoMi/publish-unit-test-result-action@v2.11.0
uses: EnricoMi/publish-unit-test-result-action@v2.18.0
if: always()
with:
check_name: Test Results ${{ inputs.result_affix }}
Expand All @@ -86,7 +86,7 @@ jobs:
- name: Upload Test Results
if: success() || failure()
uses: actions/upload-artifact@v3.1.3
uses: actions/upload-artifact@v4
with:
name: test_reports ${{ inputs.result_affix }}
path: |
Expand All @@ -95,7 +95,7 @@ jobs:
${{ inputs.cmake_path }}/build/**/regression/output_files/*.bin
- name: Configure GitHub Pages
uses: actions/configure-pages@v3.0.6
uses: actions/configure-pages@v5

- name: Generate Code Coverage Results Summary
if: (!inputs.skip_coverage)
Expand All @@ -115,7 +115,7 @@ jobs:
- name: Create CheckRun for Code Coverage
if: ((github.event_name == 'push') || (github.event_name == 'workflow_dispatch') || (github.event.pull_request.head.repo.full_name == github.repository)) && (!inputs.skip_coverage)
uses: LouisBrunner/checks-action@v1.6.2
uses: LouisBrunner/checks-action@v2.0.0
with:
token: ${{ secrets.GITHUB_TOKEN }}
name: Code Coverage ${{ inputs.result_affix }}
Expand All @@ -141,15 +141,15 @@ jobs:
fi
- name: Upload Code Coverage Artifacts
uses: actions/upload-artifact@v3.1.3
uses: actions/upload-artifact@v4
if: (inputs.skip_deploy && !inputs.skip_coverage)
with:
name: coverage_report
name: coverage_report-${{inputs.context}}-${{inputs.job-index}}
path: ${{ inputs.cmake_path }}/coverage_report
retention-days: 1

- name: Upload Code Coverage Pages
uses: actions/upload-pages-artifact@v2.0.0
uses: actions/upload-pages-artifact@v3
if: (!inputs.skip_deploy && !inputs.skip_coverage)
with:
path: ${{ inputs.cmake_path }}/coverage_report/${{ inputs.coverage_name }}
Expand All @@ -166,13 +166,14 @@ jobs:
id-token: write

steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
if: ${{ inputs.skip_test }}
with:
name: coverage_report
pattern: coverage_report-${{inputs.context}}*
merge-multiple: true

- name: Upload Code Coverage Pages
uses: actions/upload-pages-artifact@v2.0.0
uses: actions/upload-pages-artifact@v3
if: ${{ inputs.skip_test }}
with:
path: .
Expand All @@ -184,7 +185,7 @@ jobs:

- name: Deploy GitHub Pages site
id: deployment
uses: actions/deploy-pages@v1.2.9
uses: actions/deploy-pages@v4

- name: Write Code Coverage Report URL
run: >-
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/regression_test.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
name: regression_test

# Controls when the action will run. Triggers the workflow on push or pull request
# events but only for the master branch
# events but only for the dev branch
on:
workflow_dispatch:
push:
branches: [ master ]
branches: [ dev ]
pull_request:
branches: [ master ]
branches: [ dev ]

# A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
Expand All @@ -32,4 +32,4 @@ jobs:
uses: ./.github/workflows/regression_template.yml
with:
skip_test: true
deploy_list: "ThreadX SMP"
deploy_list: "ThreadX SMP"

0 comments on commit 473f3ae

Please sign in to comment.