Skip to content

Commit 5fff553

Browse files
authored
Merge branch 'main' into fix/update-androidnativebundle-to-1.1.5
2 parents 46849f4 + 18e6fd7 commit 5fff553

File tree

3 files changed

+27
-5
lines changed

3 files changed

+27
-5
lines changed

.github/workflows/changes-in-high-risk-code.yml

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,25 @@ jobs:
4141
script: |
4242
const highRiskFiles = process.env.high_risk_code;
4343
const fileList = highRiskFiles.split(',').map(file => `- [ ] ${file}`).join('\n');
44-
github.rest.issues.createComment({
44+
45+
// Get existing comments
46+
const comments = await github.rest.issues.listComments({
4547
issue_number: context.issue.number,
4648
owner: context.repo.owner,
47-
repo: context.repo.repo,
48-
body: `### 🚨 Detected changes in high risk code 🚨 \n High-risk code has higher potential to break the SDK and may be hard to test. To prevent severe bugs, apply the rollout process for releasing such changes and be extra careful when changing and reviewing these files:\n ${fileList}`
49-
})
49+
repo: context.repo.repo
50+
});
51+
52+
// Check if we already have a high risk code comment
53+
const hasExistingComment = comments.data.some(comment =>
54+
comment.body.includes('🚨 Detected changes in high risk code 🚨')
55+
);
56+
57+
// Only create comment if we don't already have one
58+
if (!hasExistingComment) {
59+
await github.rest.issues.createComment({
60+
issue_number: context.issue.number,
61+
owner: context.repo.owner,
62+
repo: context.repo.repo,
63+
body: `### 🚨 Detected changes in high risk code 🚨 \n High-risk code has higher potential to break the SDK and may be hard to test. To prevent severe bugs, apply the rollout process for releasing such changes and be extra careful when changing and reviewing these files:\n ${fileList}`
64+
});
65+
}

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,12 @@
3636
- No user context exists
3737
- User context exists but IP address is null
3838

39+
### Dependencies
40+
41+
- Bump Android SDK from v7.16.0 to v7.17.0 ([#2408](https://github.com/getsentry/sentry-dart/pull/2408))
42+
- [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#7170)
43+
- [diff](https://github.com/getsentry/sentry-java/compare/7.16.0...7.17.0)
44+
3945
## 8.10.1
4046

4147
### Fixes

flutter/android/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ android {
6060
}
6161

6262
dependencies {
63-
api 'io.sentry:sentry-android:7.16.0'
63+
api 'io.sentry:sentry-android:7.17.0'
6464
implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
6565

6666
// Required -- JUnit 4 framework

0 commit comments

Comments
 (0)