Skip to content

Fix GitHub Actions workflow triggers for li-0.11 branch#43

Merged
robreeves merged 2 commits intolinkedin:li-0.11from
robreeves:gha_fix
Apr 3, 2026
Merged

Fix GitHub Actions workflow triggers for li-0.11 branch#43
robreeves merged 2 commits intolinkedin:li-0.11from
robreeves:gha_fix

Conversation

@robreeves
Copy link
Copy Markdown
Collaborator

@robreeves robreeves commented Apr 2, 2026

Rationale for this change

  • The GitHub Actions workflows (python-ci.yml, python-ci-docs.yml, check-md-link.yml) had their push triggers set to the main branch, which does not exist in this fork. This updates them to trigger on li-0.11, which is the default branch.
  • Backports the fix from fix: Hive catalog test failure on CPython 3.13.12 apache/iceberg-python#3043 for a Hive catalog test failure on CPython 3.13.12. The test used call_count = 0 to reset a MagicMock counter, which no longer works on CPython 3.13.12. Replaced with reset_mock().

Are these changes tested?

Yes — the test fix will be validated by the CI that now runs on this branch.

Are there any user-facing changes?

No.

Replace 'main' with 'li-0.11' in push branch triggers since li-0.11 is
the default branch for this fork and main does not exist.
Cherry-pick of apache/iceberg-python#3043. Use reset_mock() instead of
directly assigning call_count = 0, which no longer works on CPython 3.13.12.
@robreeves robreeves merged commit b78e54c into linkedin:li-0.11 Apr 3, 2026
12 checks passed
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.

1 participant