Skip to content

Commit 9653e30

Browse files
authored
Revert publishing junit-report comments (#458)
# Revert publishing junit-report comments Build on Forks is broken now, because Junit-Reports plugin can't write comments. <!-- Provide a brief summary of your changes --> ## Motivation and Context Revert [this change](https://github.com/modelcontextprotocol/kotlin-sdk/pull/442/files#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721) ## 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
1 parent 0d3494b commit 9653e30

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

.github/workflows/build.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,15 +75,13 @@ jobs:
7575
if: ${{ !cancelled() }} # always run even if the previous step fails
7676
with:
7777
annotate_only: true
78-
comment: ${{ github.repository == 'modelcontextprotocol/kotlin-sdk' }}
7978
detailed_summary: true
8079
flaky_summary: true
8180
group_suite: true
8281
include_empty_in_summary: false
8382
include_time_in_summary: true
8483
report_paths: '**/test-results/**/TEST-*.xml'
8584
truncate_stack_traces: false
86-
updateComment: ${{ github.repository == 'modelcontextprotocol/kotlin-sdk' }}
8785

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

0 commit comments

Comments
 (0)