Skip to content

infra: fix Firestore CI cron jobs - #15804

Merged
ncooke3 merged 1 commit into
mainfrom
firestore-ci-fix-10749625727554048345
Feb 12, 2026
Merged

infra: fix Firestore CI cron jobs#15804
ncooke3 merged 1 commit into
mainfrom
firestore-ci-fix-10749625727554048345

Conversation

@ncooke3

@ncooke3 ncooke3 commented Feb 12, 2026

Copy link
Copy Markdown
Member

This PR fixes two issues in the sdk.firestore.yml workflow:

  1. The spm-source-cron job was failing because it didn't select an Xcode version. I added a step to select Xcode 16.2.
  2. The pod-lib-lint-cron job was being skipped on scheduled runs because its dependency check depends on changes, which was configured to skip on schedules. I removed the skip condition from changes to ensure the dependency chain works.
  3. I also added the missing xcode matrix variable to pod-lib-lint-cron to prevent it from failing once it starts running.

PR created automatically by Jules for task 10749625727554048345 started by @ncooke3

- Add explicit Xcode selection (Xcode 16.2) to `spm-source-cron` job to fix CI failure where Xcode version was not set.
- Remove `if: github.event_name != 'schedule'` from `changes` job so that dependent jobs like `check` and `pod-lib-lint-cron` can run on scheduled workflows.
- Add `xcode: [Xcode_16.4]` to `pod-lib-lint-cron` job matrix as it was missing, which would cause the job to fail when it runs.

Co-authored-by: ncooke3 <36927374+ncooke3@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@ncooke3
ncooke3 requested a review from a team as a code owner February 12, 2026 15:07
@gemini-code-assist

Copy link
Copy Markdown
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.

@ncooke3 ncooke3 changed the title Fix Firestore CI cron jobs infra: fix Firestore CI cron jobs Feb 12, 2026
@ncooke3
ncooke3 merged commit 5c64a9e into main Feb 12, 2026
72 checks passed
@ncooke3
ncooke3 deleted the firestore-ci-fix-10749625727554048345 branch February 12, 2026 20:02
@ncooke3 ncooke3 mentioned this pull request Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants