Skip to content

Conversation

@bishoy-at-pieces
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 9, 2025 22:36
@tsavo-at-pieces tsavo-at-pieces self-requested a review December 9, 2025 22:37
Copy link
Contributor

@tsavo-at-pieces tsavo-at-pieces left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to fix missing dependencies in the project. The changes reorganize dependencies in pyproject.toml, update the poetry.lock file accordingly, and enhance the SDK update workflow to install dependencies before running the SDK patch script.

  • Removes aenum dependency and adds typing-extensions as an explicit dependency
  • Reorganizes sentry-sdk placement in the dependency list
  • Updates the GitHub workflow to install Poetry and project dependencies before running sdk_patch.py

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Removes aenum, adds typing-extensions, and reorganizes sentry-sdk in the dependencies
poetry.lock Updates lock file to reflect dependency changes, removes aenum package entry and updates content hash
.github/workflows/update-sdks.yaml Adds Poetry installation and dependency installation steps before SDK patching

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bishoy-at-pieces bishoy-at-pieces merged commit f7039b4 into main Dec 9, 2025
10 of 13 checks passed
@bishoy-at-pieces bishoy-at-pieces deleted the fix-missing-deps branch December 9, 2025 22:40
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.

3 participants