Skip to content

Conversation

cabljac
Copy link
Collaborator

@cabljac cabljac commented Aug 27, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings August 27, 2025 14:35
Copy link

changeset-bot bot commented Aug 27, 2025

⚠️ No Changeset found

Latest commit: 06b92b7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package "example-react-data-connect" depends on the ignored package "@dataconnect/default-connector", but "example-react-data-connect" is not being ignored. Please add "example-react-data-connect" to the `ignore` option.
The package "@tanstack-query-firebase/angular" depends on the ignored package "@dataconnect/default-connector", but "@tanstack-query-firebase/angular" is not being ignored. Please add "@tanstack-query-firebase/angular" to the `ignore` option.
The package "@tanstack-query-firebase/react" depends on the ignored package "@dataconnect/default-connector", but "@tanstack-query-firebase/react" is not being ignored. Please add "@tanstack-query-firebase/react" to the `ignore` option.

Copy link

docs-page bot commented Aug 27, 2025

To view this pull requests documentation preview, visit the following URL:

docs.page/invertase/tanstack-query-firebase~220

Documentation is deployed and generated using docs.page.

Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves CI workflows by optimizing build performance and consistency across GitHub Actions workflows. The changes focus on enhancing caching strategies, streamlining Java setup, and improving dependency management.

Key changes:

  • Replaced manual Java installation with actions/setup-java for consistency
  • Enhanced caching for Firebase emulators, pnpm, and turbo build metadata
  • Improved dependency installation with frozen lockfile and better cache utilization

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/tests.yaml Major workflow optimization with improved caching, Java setup, and dependency management
.github/workflows/test-react-firebase-v12.yml Streamlined Java setup and quote consistency improvements

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cabljac cabljac force-pushed the @invertase/fix-ci-perf branch 2 times, most recently from 4ee161a to 10f1c31 Compare August 27, 2025 14:46
@cabljac cabljac force-pushed the @invertase/fix-ci-perf branch from 10f1c31 to 06b92b7 Compare August 27, 2025 14:51
@cabljac cabljac merged commit 4b3483b into main Aug 27, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant