Skip to content

[FDC] init dataconnect:sdk use consistent gen SDK package names#9021

Merged
fredzqm merged 5 commits intomasterfrom
fz/sdk-package
Aug 21, 2025
Merged

[FDC] init dataconnect:sdk use consistent gen SDK package names#9021
fredzqm merged 5 commits intomasterfrom
fz/sdk-package

Conversation

@fredzqm
Copy link
Copy Markdown
Contributor

@fredzqm fredzqm commented Aug 21, 2025

Description

Scenarios Tested

Sample Commands

gemini-code-assist[bot]

This comment was marked as resolved.

gemini-code-assist[bot]

This comment was marked as off-topic.

@fredzqm fredzqm closed this Aug 21, 2025
@fredzqm fredzqm reopened this Aug 21, 2025
@fredzqm fredzqm changed the title [FDC] init dataconnect:sdk should generate consistent hard-coded package name Aug 21, 2025
Copy link
Copy Markdown
Contributor

@dconeybe dconeybe left a comment

Choose a reason for hiding this comment

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

LGTM for Kotlin

@github-project-automation github-project-automation bot moved this from Done to Approved [PR] in [Cloud] Extensions + Functions Aug 21, 2025
Copy link
Copy Markdown
Contributor

@maneesht maneesht left a comment

Choose a reason for hiding this comment

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

LGTM, but can you update the tests?

Copy link
Copy Markdown
Contributor

@aashishpatil-g aashishpatil-g left a comment

Choose a reason for hiding this comment

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

LGTM for iOS

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.62%. Comparing base (00ae2e4) to head (6033572).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9021      +/-   ##
==========================================
+ Coverage   49.55%   49.62%   +0.07%     
==========================================
  Files         673      673              
  Lines       38477    38515      +38     
  Branches     7676     7662      -14     
==========================================
+ Hits        19066    19114      +48     
+ Misses      17879    17864      -15     
- Partials     1532     1537       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@fredzqm fredzqm enabled auto-merge (squash) August 21, 2025 17:39
@fredzqm fredzqm merged commit e2978ae into master Aug 21, 2025
45 of 48 checks passed
@fredzqm fredzqm deleted the fz/sdk-package branch August 21, 2025 17:48
@github-project-automation github-project-automation bot moved this from Approved [PR] to Done in [Cloud] Extensions + Functions Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants