Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bump fastapi-events dependency #6644

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Bump fastapi-events dependency #6644

merged 1 commit into from
Jul 22, 2024

Conversation

ebr
Copy link
Member

@ebr ebr commented Jul 22, 2024

Summary

This is required to fix an issue in fastapi-events OpenTelemetry implementation. melvinkcx/fastapi-events#63. PR fix is open but not yet merged: melvinkcx/fastapi-events#64

QA Instructions

  • The diff in this version bump does not affect Invoke OSS: melvinkcx/fastapi-events@v0.11.0...v0.11.1
  • The changes in the proposed OpenTelemetry fix were tested in the Invoke hosted environment and do not affect the OSS installations either

Merge Plan

Merge anytime

Checklist

  • The PR has a short but descriptive title, suitable for a changelog

@ebr ebr requested a review from psychedelicious July 22, 2024 03:52
@github-actions github-actions bot added Root python-deps PRs that change python dependencies labels Jul 22, 2024
@hipsterusername hipsterusername merged commit d043557 into main Jul 22, 2024
14 checks passed
@hipsterusername hipsterusername deleted the bump-fastapi-events branch July 22, 2024 11:15
@psychedelicious psychedelicious mentioned this pull request Jul 25, 2024
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hipsterusername hipsterusername hipsterusername approved these changes

@lstein lstein Awaiting requested review from lstein lstein is a code owner

@blessedcoolant blessedcoolant Awaiting requested review from blessedcoolant blessedcoolant is a code owner

@psychedelicious psychedelicious Awaiting requested review from psychedelicious

Assignees
No one assigned
Labels
python-deps PRs that change python dependencies Root
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ebr @hipsterusername