Skip to content

Update Python SDK - #72

Merged
maddawik merged 1 commit into
68-update-generate-workflow-to-install-depsfrom
update-sdk
Aug 2, 2025
Merged

maddawik merged 1 commit into
68-update-generate-workflow-to-install-depsfrom
update-sdk

Conversation

@vulncheck-oss-release-bot

Copy link
Copy Markdown
Contributor

New API changes have been detected.

@vulncheck-oss-release-bot vulncheck-oss-release-bot Bot added the bot Automated PR label Aug 2, 2025
@maddawik
maddawik merged commit 2a17c94 into 68-update-generate-workflow-to-install-deps Aug 2, 2025
@maddawik
maddawik deleted the update-sdk branch August 2, 2025 20:08
maddawik added a commit that referenced this pull request Aug 2, 2025
…69)

* fix(#68): install deps in workflows, bump generator 7.12.0 -> 7.14.0

This commit resolves several problems:

1. Dependencies were not being installed pre-generate workflow (not
   found in tests when there were no new changes in our OpenAPI spec).
2. Tests were failing to run because of some dependency issues. I bumped
   OpenAPI generator CLI from 7.12.0 -> 7.14.0 which updates the minimum
   version needed for `urllib3` which resolves those failures.
3. The Makefile commands and the inline bash that was being used within
   the github workflows were not in-sync (things were `pip install`d and
   not all the `make` targets were leveraged). That's all been cleaned
   up so that `poetry` wraps every command (guaranteeing we're using the
   version we specify in the `pyproject.toml`) and that `make` targets
   are used in all of the workflows.

* Update Python SDK (#72)

[bot](2025-08-02 20:05:32) Sync SDK with OpenAPI spec

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: vulncheck-oss-release-bot[bot] <187049212+vulncheck-oss-release-bot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot Automated PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant