Skip to content

Sync Python SDK models from atlanhq/models#886

Open
atlan-ci wants to merge 1 commit intomainfrom
sync-models-20260326-134041
Open

Sync Python SDK models from atlanhq/models#886
atlan-ci wants to merge 1 commit intomainfrom
sync-models-20260326-134041

Conversation

@atlan-ci
Copy link
Copy Markdown
Collaborator

Summary

This PR syncs the generated Python SDK models from the atlanhq/models repository.

Changes

Auto-generated Python model files from Pkl type definitions.

Test Plan

  • Verify generated models are syntactically correct
  • Run existing tests against updated models

🤖 This PR was automatically created by the sync-python-sdk workflow.

Source commit: 2f09c29043e4055c23a36c10fd6cbf2632f930e7
Source ref: refs/heads/master

Co-Authored-By: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@claude
Copy link
Copy Markdown

claude bot commented Mar 26, 2026

Code Review Summary - Confidence: High

This is an auto-generated models sync PR with one critical issue.

CRITICAL ISSUE - requirements.txt Emptied:
The requirements.txt file was emptied (274 lines deleted, 0 remaining). This lock file should not be empty. It appears to be an unintended side effect of the sync workflow.

CI Status - Failing:

  • qa-checks-and-unit-tests (3.10) - FAILURE
  • v9-qa-checks-and-unit-tests (3.10) - FAILURE

Non-Model Changes Review:

  • pyatlan/client/common/transport.py (+1/-1): Only import reordering. Purely cosmetic. No security concerns.

Security Checklist (per CLAUDE.md) - All pass:

  • No api_key or impersonation tokens in log output
  • No verify=False (TLS verification not disabled)
  • No base_url validation changes

Auto-Generated Model Changes:

  • 10 new model files added (KafkaCluster, KafkaField, DataMeshDataset, AuthService, etc.)
  • 450 model files modified with updated type definitions
  • All follow consistent auto-generated patterns

Recommendation:
The emptied requirements.txt must be restored or regenerated before merging. Once that is fixed and CI passes, this PR should be safe to merge.

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