Skip to content

Conversation

@Kayrnt
Copy link
Collaborator

@Kayrnt Kayrnt commented Sep 19, 2025

  • Introduced UUID type for organization_id in APIConnection, ResultSetContext, and related models.
  • Updated handling of organization_id to accept both string and UUID formats, with validation for string UUIDs.
  • Modified StatementHandler and other components to ensure proper conversion between UUID and string representations.
  • Added tests to verify UUID handling and conversion in APIConnection and StatementRequest.
  • Created a new script for generating OpenAPI v2 client code from YAML specification.
  • Add Makefile for build and CI management
  • Update CONTRIBUTING.md to reflect Makefile usage for dependency management and testing

@Kayrnt Kayrnt force-pushed the update-openapi-classes branch from 4334f8d to 341ff9c Compare September 19, 2025 11:13
…ion and related models

- Introduced UUID type for organization_id in APIConnection, ResultSetContext, and related models.
- Updated handling of organization_id to accept both string and UUID formats, with validation for string UUIDs.
- Modified StatementHandler and other components to ensure proper conversion between UUID and string representations.
- Added tests to verify UUID handling and conversion in APIConnection and StatementRequest.
- Created a new script for generating OpenAPI v2 client code from YAML specification.
@Kayrnt Kayrnt force-pushed the update-openapi-classes branch from 341ff9c to 2108a05 Compare September 19, 2025 11:19
- Introduced a Makefile to streamline common development tasks including installation, linting, formatting, type checking, testing, and building the package.
- Updated GitHub Actions workflow to utilize the Makefile for running CI checks, enhancing consistency in the build process.
- Added a target for running Jupyter Lab within the Makefile for improved development experience.
…ement and testing

- Changed instructions from `uv` commands to `make` commands for installing dependencies, running tests, linting, formatting, and type checking.
- Added additional Makefile targets for CI checks, unit tests, and build processes to streamline development workflows.
@Kayrnt Kayrnt force-pushed the update-openapi-classes branch from 6c082d2 to 778b085 Compare September 19, 2025 11:32
@Kayrnt Kayrnt marked this pull request as ready for review September 19, 2025 13:59
@Kayrnt Kayrnt changed the title Add UUID support for organization IDs and statement IDs in APIConnect… Add UUID support for organization IDs and statement IDs in APIConnection and related models Sep 19, 2025
@bazzazzadeh bazzazzadeh merged commit f053d3a into deltastreaminc:main Sep 19, 2025
4 checks passed
Kayrnt added a commit to Kayrnt/deltastream-connector-python that referenced this pull request Sep 19, 2025
Kayrnt added a commit to Kayrnt/deltastream-connector-python that referenced this pull request Sep 19, 2025
bazzazzadeh pushed a commit that referenced this pull request Sep 19, 2025
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