Skip to content

RUM-19421: Update Gradle version to 9.8.2 - #4076

Open
0xnm wants to merge 1 commit into
nogorodnikov/rum-19405/improve-gradle-setupfrom
nogorodnikov/rum-19421/update-gradle-to-9.8.2
Open

0xnm wants to merge 1 commit into
nogorodnikov/rum-19405/improve-gradle-setupfrom
nogorodnikov/rum-19421/update-gradle-to-9.8.2

Conversation

@0xnm

@0xnm 0xnm commented Oct 29, 2026

Copy link
Copy Markdown
Member

What does this PR do?

This PR updates the Gradle version used.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm added this pull request to stack #4077 October 29, 2026 15:36
@0xnm

0xnm commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-29T15:54:31.168150Z 8cb7ef7 New commits
🔒 Security Review ✅ Completed 2026-10-29T15:55:01.663990Z 8cb7ef7 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: b080980a9e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector

Copy link
Copy Markdown

🛡️ Codex Security Review · Automatically triggered

Security review completed. No security issues were found in this pull request.

Reviewed commit: b080980a9e

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@0xnm 0xnm added the ready-for-human-review All agentic review feedback was addressed and PR is ready for the human review. label Oct 29, 2026
@0xnm
0xnm marked this pull request as ready for review October 29, 2026 15:43
@0xnm
0xnm requested review from a team as code owners October 29, 2026 15:43
kikoveiga
kikoveiga previously approved these changes Oct 29, 2026

@datadog-datadog-prod-us1 datadog-datadog-prod-us1 Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bits Code Review: PASS

More details

Gradle 9.8.2 build compatibility remains unverified: the distribution was unavailable locally, and the reviewer's CI-status request returned HTTP 403.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Bits Code Review · Commit b080980

@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Pipelines  Tests

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 1 Pipeline job failed

DataDog/dd-sdk-android | test-pyramid:legacy-integration-instrumented-latest-api — ❄️ 1 known flaky test · 🔄 may pass on retry

View more details · View in GitLab

↳ ❄️ Known flaky: verifyViewEventsOnSwipe from com.datadog.android.sdk.integration.rum.ConsentGrantedFragmentTrackingTest

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 72.06% (+0.04%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8cb7ef7 | Docs | View more details | Give us feedback!

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b080980a9e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread gradle/wrapper/gradle-wrapper.properties
@0xnm
0xnm force-pushed the nogorodnikov/rum-19421/update-gradle-to-9.8.2 branch from b080980 to 8cb7ef7 Compare October 29, 2026 15:51
@0xnm
0xnm requested a review from kikoveiga October 29, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human-review All agentic review feedback was addressed and PR is ready for the human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants