Skip to content

Commit

Permalink
Added a GHA Pull-Request Workflow (neoforged#35)
Browse files Browse the repository at this point in the history
There is also a separate test-reporting workflow to allow the reporting of JUnit test results as a Github check, even if the build is running without write-permission (i.e. the PR is made from a fork).
  • Loading branch information
shartte authored Nov 8, 2023
1 parent f6f97f5 commit 6fe2e19
Show file tree
Hide file tree
Showing 2 changed files with 70 additions and 0 deletions.
47 changes: 47 additions & 0 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
name: 'Build Pull Request'
run-name: Build for PR ${{ github.event.pull_request.number }}

on:
pull_request:
types:
- synchronize
- opened
- ready_for_review
- reopened

jobs:
build:
name: Build and Test
runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: actions/checkout@v4
with:
fetch-depth: 1000
fetch-tags: true

- name: Setup JDK
uses: actions/setup-java@v3
with:
java-version: '17'
distribution: 'microsoft'

- name: Setup Gradle
uses: gradle/gradle-build-action@v2

- name: Build
run: ./gradlew --info -s -x check build

- name: Test
run: ./gradlew --info -s check

# Always upload test results
- name: Merge Test Reports
if: success() || failure()
run: npx junit-report-merger junit.xml "**/TEST-*.xml"

- uses: actions/upload-artifact@v3
if: success() || failure()
with:
name: test-results
path: junit.xml
23 changes: 23 additions & 0 deletions .github/workflows/test-report.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: 'Test Report'
on:
workflow_run:
workflows: ['Build Pull Request']
types:
- completed
permissions:
contents: read
actions: read
checks: write
jobs:
report:
runs-on: ubuntu-latest
steps:
- uses: neoforged/action-test-reporter@v1
with:
artifact: test-results
name: Test Report
path: '**/*.xml'
reporter: java-junit
# This should not affect the result of the check created
# for the origin PR. This just affects this post-processing job itself.
continue-on-error: true

0 comments on commit 6fe2e19

Please sign in to comment.