Skip to content

[python] run Petstore CI for lazy imports#24093

Merged
wing328 merged 1 commit into
OpenAPITools:masterfrom
tamird:codex/python-lazy-imports-ci-path
Jun 22, 2026
Merged

[python] run Petstore CI for lazy imports#24093
wing328 merged 1 commit into
OpenAPITools:masterfrom
tamird:codex/python-lazy-imports-ci-path

Conversation

@tamird

@tamird tamird commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

The Python Petstore workflow tests python-lazyImports in its matrix,
but its pull-request path filter does not include that directory. A
change confined to the lazy-import sample therefore receives no Python
Petstore coverage.

Add the missing path so the workflow runs whenever that sample changes.

@tamird

tamird commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Extracted from #24092.

@tamird tamird marked this pull request as ready for review June 22, 2026 14:33

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Re-trigger cubic

The Python Petstore workflow tests python-lazyImports in its matrix, but
its pull-request path filter does not include that directory. A change
confined to the lazy-import sample therefore receives no Python
Petstore coverage.

Add the missing path so the workflow runs whenever that sample changes.
@tamird tamird force-pushed the codex/python-lazy-imports-ci-path branch from d86a440 to 7f733f6 Compare June 22, 2026 14:34
@wing328 wing328 merged commit 97cb5b3 into OpenAPITools:master Jun 22, 2026
21 of 31 checks passed
@wing328 wing328 added this to the 7.24.0 milestone Jun 22, 2026
@tamird tamird deleted the codex/python-lazy-imports-ci-path branch June 22, 2026 14:54
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.

2 participants