Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

fix: replace empty dict with None to avoid mutable default arguments#1416

Merged
sycai merged 3 commits intomainfrom
sycai_fix
Feb 21, 2025
Merged

fix: replace empty dict with None to avoid mutable default arguments#1416
sycai merged 3 commits intomainfrom
sycai_fix

Conversation

@sycai
Copy link
Copy Markdown
Contributor

@sycai sycai commented Feb 21, 2025

No description provided.

@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels Feb 21, 2025
@sycai sycai marked this pull request as ready for review February 21, 2025 21:11
@sycai sycai requested review from a team and shuoweil February 21, 2025 21:11
@sycai sycai requested review from tswast and removed request for shuoweil February 21, 2025 21:11
@sycai sycai changed the title fix: replace empty dict wit None to avoid mutable default arguments fix: replace empty dict with None to avoid mutable default arguments Feb 21, 2025
@sycai sycai enabled auto-merge (squash) February 21, 2025 21:16
@sycai sycai merged commit fa4e3ad into main Feb 21, 2025
@sycai sycai deleted the sycai_fix branch February 21, 2025 22:43
@release-please release-please bot mentioned this pull request Feb 21, 2025
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@tswast tswast tswast approved these changes

@TrevorBergeron TrevorBergeron TrevorBergeron approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@tswast tswast

Labels

api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. size: xs Pull request size is extra small.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sycai @tswast @TrevorBergeron