Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update GitHub Actions #496

Merged
merged 5 commits into from
Aug 30, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions .github/workflows/android.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
steps:
- uses: actions/checkout@v4
- name: Get modern CMake and Ninja
uses: "lukka/[email protected].1"
uses: "lukka/[email protected].2"

# Do this before building aar since it affects the version
- name: Touch SNAPSHOT marker file
Expand All @@ -36,7 +36,7 @@ jobs:
run: "./maintainer-scripts/build-aar.sh"

- name: Upload bare AAR
uses: "actions/upload-artifact@v3"
uses: "actions/upload-artifact@v4"
with:
name: AAR and POM
path: openxr_loader_for_android*
Expand All @@ -47,12 +47,13 @@ jobs:
steps:
- uses: "actions/checkout@v4"
- name: "Get modern CMake and Ninja"
uses: "lukka/[email protected].1"
uses: "lukka/[email protected].2"
- name: "set up JDK 11"
uses: "actions/setup-java@v4"
with:
java-version: "11"
distribution: "temurin"
# Also performs Gradle wrapper validation
- uses: "gradle/actions/setup-gradle@v4"

# HelloXR OpenGLES version
Expand All @@ -61,7 +62,7 @@ jobs:

- name: Upload HelloXR OpenGLES (debug) for Android
if: "!inputs.release"
uses: "actions/upload-artifact@v3"
uses: "actions/upload-artifact@v4"
with:
name: HelloXR OpenGLES Android
path: "src/tests/hello_xr/build/outputs/apk/OpenGLES/debug/*.apk"
2 changes: 1 addition & 1 deletion .github/workflows/check_clang_format_and_codespell.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
if: ${{ failure() }}
- name: Publish diff
if: ${{ failure() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
path: ${{ runner.temp }}/clang-format.patch
name: clang-format-changes
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/gradle-wrapper-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
#
# SPDX-License-Identifier: CC0-1.0

# Probably no longer needed as long as we have another Gradle build.
name: Validate Gradle Wrapper

on:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/msvc-build-preset.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright 2021-2023, Collabora, Ltd.
# Copyright 2021-2024, Collabora, Ltd.
# SPDX-License-Identifier: CC0-1.0

on:
Expand Down Expand Up @@ -34,7 +34,7 @@ jobs:
lfs: true

- name: Get modern CMake and Ninja
uses: lukka/[email protected].1
uses: lukka/[email protected].2

- name: Add msbuild to PATH
uses: microsoft/setup-msbuild@v2
Expand All @@ -57,7 +57,7 @@ jobs:
run: "cmake --build $env:RUNNER_TEMP --parallel --config ${{ inputs.buildType }} --target install"

- name: Upload Artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
if: inputs.artifactName
with:
name: "${{ inputs.artifactName }}"
Expand Down
14 changes: 7 additions & 7 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright 2021-2022, Collabora, Ltd.
# Copyright 2021-2024, Collabora, Ltd.
# SPDX-License-Identifier: CC0-1.0

name: Build release artifacts
Expand All @@ -8,13 +8,13 @@ on:
workflow_dispatch:

jobs:
android:
uses: ./.github/workflows/android.yml
with:
release: true
# Currently Android artifacts are generated and signed internally before publication to GitHub.
# android:
# uses: ./.github/workflows/android.yml
# with:
# release: true

msvc-build:
uses: ./.github/workflows/windows-matrix.yml
with:
organizeAndRelease: true # TODO

organizeAndRelease: true
6 changes: 3 additions & 3 deletions .github/workflows/windows-matrix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,15 @@ jobs:
- uses: "actions/checkout@v4"

- name: Retrieve artifacts
uses: "actions/download-artifact@v3"
uses: "actions/download-artifact@v4"
with:
path: artifacts

- name: Organize artifacts
run: 'python .github/scripts/organize_windows_artifacts.py "${{ github.workspace }}" "${{ github.workspace }}/openxr_loader"'

- name: Upload combined artifact
uses: "actions/upload-artifact@v3"
uses: "actions/upload-artifact@v4"
with:
name: openxr_loader_windows
path: "${{ github.workspace }}/openxr_loader"
Expand All @@ -78,7 +78,7 @@ jobs:
run: 'nuget pack "${{ github.workspace }}/openxr_loader_staging/OpenXR.Loader.nuspec" -OutputDirectory "${{ github.workspace }}/nuget"'

- name: Upload NuGet artifact
uses: "actions/upload-artifact@v3"
uses: "actions/upload-artifact@v4"
with:
name: NuGet
path: "${{ github.workspace }}/nuget"