Skip to content

Conversation

@cherylEnkidu
Copy link
Contributor

@cherylEnkidu cherylEnkidu commented Dec 19, 2025

All public APIs are located under Firestore/Swift/Source/SwiftAPI/Pipeline. Thanks to the technical writer for reviewing them.

cherylEnkidu and others added 30 commits March 13, 2025 10:18
Co-authored-by: wu-hui <[email protected]>
Co-authored-by: Nick Cooke <[email protected]>
Co-authored-by: wu-hui <[email protected]>
Co-authored-by: Nick Cooke <[email protected]>
Co-authored-by: wu-hui <[email protected]>
Co-authored-by: Nick Cooke <[email protected]>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@gemini-code-assist
Copy link
Contributor

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

@google-cla
Copy link

google-cla bot commented Dec 19, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Contributor

@wu-hui wu-hui left a comment

Choose a reason for hiding this comment

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

Some ultranits.

@cherylEnkidu cherylEnkidu assigned nk-1983 and unassigned egilmorez Dec 22, 2025
@wu-hui wu-hui requested a review from nk-1983 December 23, 2025 00:22
Copy link

@nk-1983 nk-1983 left a comment

Choose a reason for hiding this comment

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

LGTM

@google-oss-bot
Copy link
Collaborator

Danger has errored

[!] Invalid Dangerfile file: POST https://api.github.com/repos/firebase/firebase-ios-sdk/issues/15625/labels: 403 - Must have admin rights to Repository. // See: https://docs.github.com/rest/issues/labels#add-labels-to-an-issue. Updating the Danger gem might fix the issue. Your Danger version: 8.4.5, latest Danger version: 9.5.3

 #  from Dangerfile:43
 #  -------------------------------------------
 #    repo_name = "firebase/firebase-ios-sdk"
 >    github.api.add_labels_to_an_issue(repo_name, issue_number, label_array)
 #  end
 #  -------------------------------------------

Generated by 🚫 Danger

@cherylEnkidu cherylEnkidu enabled auto-merge (squash) December 23, 2025 20:00
@cherylEnkidu cherylEnkidu merged commit 1feabe5 into main Dec 23, 2025
65 checks passed
@cherylEnkidu cherylEnkidu deleted the feat/pipeline/private-preview branch December 23, 2025 20:00
@paulb777
Copy link
Member

Looks like the "Bridge" types introduced in this PR broke CI - https://github.com/firebase/firebase-ios-sdk/actions/runs/20474176486/job/58835500791?pr=15638

@ncooke3
Copy link
Member

ncooke3 commented Dec 26, 2025

Looks like the "Bridge" types introduced in this PR broke CI - https://github.com/firebase/firebase-ios-sdk/actions/runs/20474176486/job/58835500791?pr=15638

More examples in the nightly testing report: #15607 (comment)

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.