Skip to content

Conversation

@cherylEnkidu
Copy link
Contributor

No description provided.

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]>
@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.


- name: Build and test
run: |
export EXPERIMENTAL_MODE=true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks this this flag was added by mistake in MIEQ PR: 8d66bdc The intention of the given commit is to use latest emulator. So I remove these export EXPERIMENTAL_MODE=true here.

platforms: iOS
buildonly_platforms: iOS

check-firestore-internal-public-headers:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove these check since right now the not all of public APIs of objective C files are intend to use by customers.

# Unreleased
- [feature] Add `Pipeline` support.
- [fixed] Fixed an issue where the returned object in transaction blocks could not
pass across actor boundaries in Swift 6 (#15467).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

flags |= SQLITE_OPEN_FILEPROTECTION_NONE;
#endif
int result = sqlite3_open_v2([path UTF8String], &self -> _database, flags, NULL);
int result = sqlite3_open_v2([path UTF8String], &self->_database, flags, NULL);
Copy link
Contributor

Choose a reason for hiding this comment

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

This is likely from clang-format?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I will revert this one

@@ -1,3 +1,8 @@
# Unreleased
- [feature] Add `Pipeline` support.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe call out it is a public preview feature.

- (void)testSpecTests {
if ([self isTestBaseClass]) return;

// LogSetLevel(firebase::firestore::util::kLogLevelDebug);
Copy link
Contributor

Choose a reason for hiding this comment

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

Just delete?

} else {
++testSkipCount;
NSLog(@" [SKIPPED] Spec test: %@", name);
// NSLog(@" [SKIPPED] Spec test: %@", name);
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like we should bring it back?

@@ -1,5 +1,5 @@
/*
* Copyright 2017 Google
* Copyright 2025 Google LLC
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo?

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.