Skip to content

Revert "Update headers to track vertex ai/developer usage in kotlin (… #6949

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

Merged
merged 1 commit into from
May 9, 2025

Conversation

daymxn
Copy link
Member

@daymxn daymxn commented May 8, 2025

This reverts commit #6944.

The change was meant to be for firebase-ai, which we corrected with #6948.

@daymxn daymxn requested a review from VinayGuthal May 8, 2025 20:16
@daymxn daymxn self-assigned this May 8, 2025
@daymxn daymxn enabled auto-merge (squash) May 8, 2025 20:16
Copy link
Contributor

github-actions bot commented May 8, 2025

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

No changes between base commit (2a17764) and merge commit (3f6a703).

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/gzQH92TXWy.html

Copy link
Contributor

github-actions bot commented May 8, 2025

Test Results

 18 files   -  4   18 suites   - 4   35s ⏱️ -1s
 88 tests  - 12   88 ✅  - 12  0 💤 ±0  0 ❌ ±0 
176 runs   - 24  176 ✅  - 24  0 💤 ±0  0 ❌ ±0 

Results for commit da9c8bb. ± Comparison against base commit 2a17764.

This pull request removes 100 and adds 88 tests. Note that renamed tests count towards both.
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ citation parsed correctly
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ image rejected
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ long reply
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ prompt blocked for safety
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ short reply
com.google.firebase.ai.DevAPIStreamingSnapshotTests ‑ stopped for recitation
com.google.firebase.ai.DevAPIUnarySnapshotTests ‑ citation returns correctly
com.google.firebase.ai.DevAPIUnarySnapshotTests ‑ invalid api key
com.google.firebase.ai.DevAPIUnarySnapshotTests ‑ long reply
com.google.firebase.ai.DevAPIUnarySnapshotTests ‑ response blocked for safety
…
com.google.firebase.vertexai.GenerativeModelTesting ‑ exception thrown when using invalid location
com.google.firebase.vertexai.GenerativeModelTesting ‑ system calling in request
com.google.firebase.vertexai.SchemaTests ‑ basic schema declaration
com.google.firebase.vertexai.SchemaTests ‑ full schema declaration
com.google.firebase.vertexai.SerializationTests ‑ test Candidate serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test GenerateContentResponse serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test Schema serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test countTokensResponse serialization as Json
com.google.firebase.vertexai.SerializationTests ‑ test modalityTokenCount serialization as Json
com.google.firebase.vertexai.VertexAIStreamingSnapshotTests ‑ citation parsed correctly
…

@google-oss-bot
Copy link
Contributor

@daymxn daymxn merged commit eee76b4 into main May 9, 2025
35 checks passed
@daymxn daymxn deleted the daymon-fix-headers branch May 9, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants