Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Aug 5, 2021

Closes #414.

@tseaver tseaver requested review from a team, craiglabenz and crwilcox August 5, 2021 20:04
@tseaver tseaver requested a review from a team as a code owner August 5, 2021 20:04
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 5, 2021
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/python-firestore API. label Aug 5, 2021
@crwilcox
Copy link
Contributor

@tseaver is this just open to run CI or is the goal to merge this in? Is there any concern about getting other prereleases?

@tseaver
Copy link
Contributor Author

tseaver commented Aug 11, 2021

@crwilcox

Is this just open to run CI or is the goal to merge this in? Is there any concern about getting other prereleases?

I meant it to be merged, so that we get notified ASAP when a pre-release dependency breaks our tests.

@tseaver tseaver merged commit a1e9a16 into master Aug 12, 2021
@tseaver tseaver deleted the 414-test-against-prerelease-deps-under-3.9 branch August 12, 2021 02:16
tseaver added a commit that referenced this pull request Aug 17, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 18, 2021
Reverts #415

Consensus from today's meeting is that testing prereleases of third-party dependencies needs to happen outside the normal `presubmit` path.
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Nov 24, 2025
Reverts googleapis/python-firestore#415

Consensus from today's meeting is that testing prereleases of third-party dependencies needs to happen outside the normal `presubmit` path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the googleapis/python-firestore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests against pre-release versions of dependencies

2 participants