Skip to content

Commit 0d3494b

Browse files
authored
Update GitHub Actions workflow (#452)
Update GitHub Actions workflow - Rename `validate-pr` job to `build` - Update `comment` and `updateComment` parameters to be false on forks <!-- Provide a brief summary of your changes --> ## Motivation and Context mikepenz/action-junit-report GH Action can't create/update comments on forks ## How Has This Been Tested? CI ## Breaking Changes No ## Types of changes <!-- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update ## Checklist <!-- Go over all the following points, and put an `x` in all the boxes that apply. --> - [x] I have read the [MCP Documentation](https://modelcontextprotocol.io) - [x] My code follows the repository's style guidelines - [x] New and existing tests pass locally - [ ] I have added appropriate error handling - [ ] I have added or updated documentation as needed ## Additional context <!-- Add any other context, implementation notes, or design decisions -->
1 parent 274089a commit 0d3494b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/build.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ permissions:
1717
pull-requests: write # only required if `comment: true` was enabled
1818

1919
jobs:
20-
validate-pr:
20+
build:
2121
runs-on: macos-latest-xlarge
22-
name: Validate PR
22+
name: Build
2323
timeout-minutes: 20
2424
env:
2525
JAVA_OPTS: "-Xmx8g -Dfile.encoding=UTF-8 -Djava.awt.headless=true -Dkotlin.daemon.jvm.options=-Xmx6g"
@@ -75,15 +75,15 @@ jobs:
7575
if: ${{ !cancelled() }} # always run even if the previous step fails
7676
with:
7777
annotate_only: true
78-
comment: true
78+
comment: ${{ github.repository == 'modelcontextprotocol/kotlin-sdk' }}
7979
detailed_summary: true
8080
flaky_summary: true
8181
group_suite: true
8282
include_empty_in_summary: false
8383
include_time_in_summary: true
8484
report_paths: '**/test-results/**/TEST-*.xml'
8585
truncate_stack_traces: false
86-
updateComment: true
86+
updateComment: ${{ github.repository == 'modelcontextprotocol/kotlin-sdk' }}
8787

8888
- name: Disable Auto-Merge on Fail
8989
if: failure() && github.event_name == 'pull_request'

0 commit comments

Comments
 (0)