Skip to content

chore: skip sqlalchemy-bigquery test to unblock CI - #17288

Merged
ohmayr merged 6 commits into
mainfrom
skip-sqlalchemy-test
Jun 12, 2026
Merged

chore: skip sqlalchemy-bigquery test to unblock CI#17288
ohmayr merged 6 commits into
mainfrom
skip-sqlalchemy-test

Conversation

@ohmayr

@ohmayrohmayr commented May 28, 2026•
edited by parthea
Loading

Copy link
Copy Markdown
Contributor

skip sqlalchemy-bigquery test to keep builds green. See: #17287.

Releases will be blocked for sqlalchemy-bigquery until this test is fixed. See #17289

@ohmayr
ohmayr requested review from a team as code ownersMay 28, 2026 15:56
@ohmayr
ohmayr requested review from tswast and removed request for a teamMay 28, 2026 15:56
gemini-code-assist[bot]
gemini-code-assistBot reviewed May 28, 2026
View reviewed changes

@gemini-code-assistgemini-code-assistBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds a skip decorator and a TODO comment to the test_geoalchemy2_core test due to a CI failure. The reviewer suggested correcting the skip reason's formatting by fixing an unclosed bracket, providing the full URL, and updating the TODO link to use HTTPS.

Comment threadpackages/sqlalchemy-bigquery/tests/unit/test_geography.py Outdated
@ohmayr
ohmayr merged commit 4d3447d into mainJun 12, 2026
32 checks passed
@ohmayr
ohmayr deleted the skip-sqlalchemy-test branch June 12, 2026 19:34
chalmerlowe pushed a commit that referenced this pull request Jun 25, 2026
@ohmayr@chalmerlowe
chore: skip sqlalchemy-bigquery test to unblock CI (#17288)
b152ca2
skip sqlalchemy-bigquery test to keep builds green. See:
#17287.
Releases will be blocked for `sqlalchemy-bigquery` until this test is
fixed. See #17289
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tswasttswasttswast approved these changes
@partheapartheaparthea approved these changes
+1 more reviewer
@gemini-code-assistgemini-code-assist[bot]gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ohmayr@tswast@parthea