This is an automated email from the ASF dual-hosted git repository.
av pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git
The following commit(s) were added to refs/heads/master by this push:
new d48adb18119 IGNITE-28204 Fine tuning `protected classes` CI job
(#12880)
d48adb18119 is described below
commit d48adb1811951bb8411d8faa8a3755e40a9c8a70
Author: Alexey Abashev <[email protected]>
AuthorDate: Wed Mar 11 19:24:24 2026 +0300
IGNITE-28204 Fine tuning `protected classes` CI job (#12880)
---
.github/workflows/check-protected-classes.yml | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/.github/workflows/check-protected-classes.yml
b/.github/workflows/check-protected-classes.yml
index ce6538e234f..1b0c23a4c9e 100644
--- a/.github/workflows/check-protected-classes.yml
+++ b/.github/workflows/check-protected-classes.yml
@@ -14,8 +14,8 @@
# limitations under the License.
name: Check Protected Classes
-on:
- pull_request:
+
+on: pull_request_target
concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
@@ -25,6 +25,9 @@ jobs:
check-protected-classes:
runs-on: ubuntu-latest
name: Check protected classes
+ permissions:
+ issues: write
+ pull-requests: write
steps:
- uses: actions/checkout@v6
with:
@@ -66,9 +69,9 @@ jobs:
const fs = require('fs');
const hits = fs.readFileSync('/tmp/protected-hits.txt',
'utf8').trim();
const body = [
- '## Protected Classes Review Required',
+ '## Possible compatibility issues. Please, check rolling upgrade
cases',
'',
- 'This PR modifies protected classes (with @Order annotation).',
+ 'This PR modifies protected classes (with **Order**
annotation).',
'Changes to these classes can break rolling upgrade
compatibility.',
'',
'**Affected files:**',
@@ -82,7 +85,7 @@ jobs:
issue_number: context.issue.number,
});
- const existing = comments.find(c => c.body.includes('Protected
Classes Review Required'));
+ const existing = comments.find(c => c.body.includes('Possible
compatibility issues. Please, check rolling upgrade cases'));
if (existing) {
await github.rest.issues.updateComment({
@@ -109,5 +112,5 @@ jobs:
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: context.issue.number,
- labels: ['protected-classes'],
+ labels: ['compatibility'],
});