-
Notifications
You must be signed in to change notification settings - Fork 607
Removed deprecated public constructor KeyValueBuilder #6924
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Javadoc Changes:--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/crashlytics/KeyValueBuilder.html 2025-04-30 20:59:47.901144582 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/crashlytics/KeyValueBuilder.html 2025-04-30 20:57:18.832816678 +0000
@@ -21,27 +21,6 @@
</colgroup>
<thead>
<tr>
- <th colspan="100%"><h3>Public constructors</h3></th>
- </tr>
- </thead>
- <tbody class="list">
- <tr>
- <td>
- <div><code><span><del><a href="/docs/reference/android/com/google/firebase/crashlytics/KeyValueBuilder.html#KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/crashlytics/FirebaseCrashlytics.html">FirebaseCrashlytics</a> crashlytics)</code></div>
- <p><strong>This method is deprecated.</strong> Do not construct this directly.</p>
- </td>
- </tr>
- </tbody>
- </table>
- </div>
- <div class="devsite-table-wrapper">
- <table class="responsive">
- <colgroup>
- <col width="40%">
- <col>
- </colgroup>
- <thead>
- <tr>
<th colspan="100%"><h3>Public methods</h3></th>
</tr>
</thead>
@@ -92,14 +71,6 @@
</table>
</div>
<div class="list">
- <h2>Public constructors</h2>
- <div class="api-item"><a name="KeyValueBuilder-com.google.firebase.crashlytics.FirebaseCrashlytics-"></a><a name="keyvaluebuilder"></a>
- <h3 class="api-name" id="KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</h3>
- <pre class="api-signature no-pretty-print">public <span><del><a href="/docs/reference/android/com/google/firebase/crashlytics/KeyValueBuilder.html#KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/crashlytics/FirebaseCrashlytics.html">FirebaseCrashlytics</a> crashlytics)</pre>
- <aside class="caution"><strong>This method is deprecated.</strong><br>Do not construct this directly. Use `setCustomKeys` instead. To be removed in the next major release.</aside>
- </div>
- </div>
- <div class="list">
<h2>Public methods</h2>
<div class="api-item"><a name="key(kotlin.String, kotlin.Boolean)"></a><a name="key-kotlin.String-kotlin.Boolean-"></a><a name="key"></a>
<h3 class="api-name" id="key(kotlin.String,kotlin.Boolean)">key</h3> --- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/crashlytics/KeyValueBuilder.html 2025-04-30 20:59:47.899144590 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/crashlytics/KeyValueBuilder.html 2025-04-30 20:57:18.819816741 +0000
@@ -21,27 +21,6 @@
</colgroup>
<thead>
<tr>
- <th colspan="100%"><h3>Public constructors</h3></th>
- </tr>
- </thead>
- <tbody class="list">
- <tr>
- <td>
- <div><code><span><del><a href="/docs/reference/kotlin/com/google/firebase/crashlytics/KeyValueBuilder.html#KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</a></del></span>(crashlytics: <a href="/docs/reference/kotlin/com/google/firebase/crashlytics/FirebaseCrashlytics.html">FirebaseCrashlytics</a>)</code></div>
- <p><strong>This function is deprecated.</strong> Do not construct this directly.</p>
- </td>
- </tr>
- </tbody>
- </table>
- </div>
- <div class="devsite-table-wrapper">
- <table class="responsive">
- <colgroup>
- <col width="40%">
- <col>
- </colgroup>
- <thead>
- <tr>
<th colspan="100%"><h3>Public functions</h3></th>
</tr>
</thead>
@@ -92,14 +71,6 @@
</table>
</div>
<div class="list">
- <h2>Public constructors</h2>
- <div class="api-item"><a name="KeyValueBuilder-com.google.firebase.crashlytics.FirebaseCrashlytics-"></a><a name="keyvaluebuilder"></a>
- <h3 class="api-name" id="KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</h3>
- <pre class="api-signature no-pretty-print"><span><del><a href="/docs/reference/kotlin/com/google/firebase/crashlytics/KeyValueBuilder.html#KeyValueBuilder(com.google.firebase.crashlytics.FirebaseCrashlytics)">KeyValueBuilder</a></del></span>(crashlytics: <a href="/docs/reference/kotlin/com/google/firebase/crashlytics/FirebaseCrashlytics.html">FirebaseCrashlytics</a>)</pre>
- <aside class="caution"><strong>This function is deprecated.</strong><br>Do not construct this directly. Use `setCustomKeys` instead. To be removed in the next major release.</aside>
- </div>
- </div>
- <div class="list">
<h2>Public functions</h2>
<div class="api-item"><a name="key(kotlin.String, kotlin.Boolean)"></a><a name="key-kotlin.String-kotlin.Boolean-"></a><a name="key"></a>
<h3 class="api-name" id="key(kotlin.String,kotlin.Boolean)">key</h3> |
Vertex AI Mock Responses Check
|
Coverage Report 1Affected ProductsTest Logs |
The public api surface has changed for the subproject firebase-crashlytics: Please update the api.txt files for the subprojects being affected by this change by running ./gradlew ${subproject}:generateApiTxtFile. Also perform a major/minor bump accordingly. |
ef60d63
to
1892a2d
Compare
1892a2d
to
eb1bb02
Compare
The public api surface has changed for the subproject firebase-crashlytics: Please update the api.txt files for the subprojects being affected by this change by running ./gradlew ${subproject}:generateApiTxtFile. Also perform a major/minor bump accordingly. |
eb1bb02
to
19d9fed
Compare
Test Results 8 files - 14 8 suites - 14 20s ⏱️ +6s Results for commit 19d9fed. ± Comparison against base commit 2b23887. This pull request removes 98 and adds 22 tests. Note that renamed tests count towards both.
|
Size Report 1Affected Products
Test Logs |
Startup Time Report 1Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS. Notes
Startup Times
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment left only to prevent accidental merging the PR
prepare for next major bump
Original change:#6536