Skip to content

Commit

Permalink
Merge branch 'main' into feat/screenshot-debounce
Browse files Browse the repository at this point in the history
  • Loading branch information
denrase authored Nov 18, 2024
2 parents e96bb8d + 18e6fd7 commit 5a9bcac
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 5 deletions.
24 changes: 20 additions & 4 deletions .github/workflows/changes-in-high-risk-code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,25 @@ jobs:
script: |
const highRiskFiles = process.env.high_risk_code;
const fileList = highRiskFiles.split(',').map(file => `- [ ] ${file}`).join('\n');
github.rest.issues.createComment({
// Get existing comments
const comments = await github.rest.issues.listComments({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
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}`
})
repo: context.repo.repo
});
// Check if we already have a high risk code comment
const hasExistingComment = comments.data.some(comment =>
comment.body.includes('🚨 Detected changes in high risk code 🚨')
);
// Only create comment if we don't already have one
if (!hasExistingComment) {
await github.rest.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
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}`
});
}
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,12 @@
- No user context exists
- User context exists but IP address is null

### Dependencies

- Bump Android SDK from v7.16.0 to v7.17.0 ([#2408](https://github.com/getsentry/sentry-dart/pull/2408))
- [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#7170)
- [diff](https://github.com/getsentry/sentry-java/compare/7.16.0...7.17.0)

## 8.10.1

### Fixes
Expand Down
2 changes: 1 addition & 1 deletion flutter/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ android {
}

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

// Required -- JUnit 4 framework
Expand Down

0 comments on commit 5a9bcac

Please sign in to comment.